-
Notifications
You must be signed in to change notification settings - Fork 447
[FIRRTL] Enable access to IMDCE at smaller granularities #10055
Copy link
Copy link
Open
Labels
FIRRTLInvolving the `firrtl` dialectInvolving the `firrtl` dialectReducerRelated to `circt-reduce`Related to `circt-reduce`enhancementNew feature or requestNew feature or request
Description
I've turned on IMDCE in circt-reduce for FIRRTL. However, this is a very heavy hammer. E.g., I have a circuit with dead code. However, when I run it through IMDCE it deletes essentially everything. In order for this to be effective in the reducer, there needs to be a more targeted way of applying this.
There are a couple of features that would help with this:
- Being able to run IMDCE on a per-module basis.
- Being able to run IMDCE on only specific signals. E.g., on a per-port basis.
Likely this would entail breaking the main IMDCE pass into smaller units (or utilities) that could be reused.
There are alternatives here which involve selective application of inner symbols to block removal. However, that's a bit rough.
h/t @uenoku for the idea.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
FIRRTLInvolving the `firrtl` dialectInvolving the `firrtl` dialectReducerRelated to `circt-reduce`Related to `circt-reduce`enhancementNew feature or requestNew feature or request
Type
Fields
Give feedbackNo fields configured for issues without a type.