Open JulianKemmerer opened 1 year ago
Gah, made this issue thinking it shouldnt block whats in #86
But in testing it seems the rework that makes #86 almost done does need some ~optimization (not funcitonal) changes to meet timing in some of the old tests... so I guess addressing this issue too maybe...
Actually will probably not require this change right away with #86 and can instead do at leisure when using results of combined #86 and instance array wires branch to do cool things
Want a pass that can combine sequential-looking FSM states that occur in the same cycle into one/as few combintatorial state transitions as possible (watching out for states that can't be jumped to / combined to others)
Then the parallel state grouping in
C_AST_FSM_FUNDEF_BODY_TO_LOGIC
andGET_GROUPED_STATE_TRANSITIONS
seems like it has room for improvement, especially regarding ordering of inlined subroutine states vs single instance fsm module connection entry+FSM+exit triple state sequence.