-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[When] Split when blocks with cycles #1309
Conversation
Codecov ReportPatch coverage:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## fix-1253 #1309 +/- ##
============================================
+ Coverage 85.90% 86.08% +0.17%
============================================
Files 164 170 +6
Lines 17637 17818 +181
============================================
+ Hits 15151 15338 +187
+ Misses 2486 2480 -6
☔ View full report in Codecov by Sentry. |
fb60543
to
9d4243c
Compare
157f719
to
1219c59
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Final minor comment
This reverts commit 512c7e1.
This reverts commit 512c7e1.
Fixes #1269
Adds a pass that splits when blocks with a cyclic dependency.
Note the generated verilog can have empty blocks since some of the assignments are being emitted in a split block, but this seems to be okay for verilator. Since this is likely to be an uncommon case, I don't think we should worry too much about trying to cleanup the generated verilog.