rohd
rohd copied to clipboard
Improving the `StateMachine` view in `vcd`
Motivation
When viewing vcd StateMachine is split into sequential and combinational, which introduces a bit of chaos due to the lack of context. It is better to let the user immediately see which module these blocks belong to.

Desired solution
- Generate the
statemachinemodule, inside which thesequentialandcombinationalblocks will be located. - Add the ability to set a name for
StateMachine(the default value can bestatemachine).
Alternatives considered
No response
Additional details
No response
Great suggestion. I think something similar might be nice for Pipeline as well.
As a note, it will require a little overhead in the implementation to properly generate inputs and outputs to the new module. For example, instead of passing List<Conditional>s directly into the Combinational blocks, they will have to be recreated using inputs and outputs of the module.