You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
DontTouchAnnotations are not getting deleted after the SFC pass, which results in MFC errors. This is because MFC will try to perform passes correlated to DontTouchAnnotations but the related variables may be missing(SFC could have renamed it or sth).
Currently, we are addressing this by deleting all the corresponding annotations by using jq inside common.mk (#1239).
In the long term, we want a cleaner way of doing this (e.g., fix the SFC to delete the annotations after they are used).
Another related issue is that MFC cannot compile designs containing Fixed & Interval types. At the moment, we are using SFC to compile CHIRRTL into LowFIRRTL & MFC to compile LowFIRRTL to Verilog when we find Fixed types inside the design (we perform grep Fixed inside common.mk). For the SFC + MFC compilation steps to work more elegantly, we need to rewrite Fixed types (and Interval types).
Background Work
Feature Description
DontTouchAnnotation
s are not getting deleted after the SFC pass, which results in MFC errors. This is because MFC will try to perform passes correlated toDontTouchAnnotation
s but the related variables may be missing(SFC could have renamed it or sth).Currently, we are addressing this by deleting all the corresponding annotations by using
jq
insidecommon.mk
(#1239).In the long term, we want a cleaner way of doing this (e.g., fix the SFC to delete the annotations after they are used).
Another related issue is that MFC cannot compile designs containing
Fixed
&Interval
types. At the moment, we are using SFC to compileCHIRRTL
intoLowFIRRTL
& MFC to compileLowFIRRTL
toVerilog
when we findFixed
types inside the design (we performgrep Fixed
insidecommon.mk
). For the SFC + MFC compilation steps to work more elegantly, we need to rewriteFixed
types (andInterval
types).Motivating Example
Look at #1239.
The text was updated successfully, but these errors were encountered: