-
Notifications
You must be signed in to change notification settings - Fork 276
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
[FIRRTL][CheckCombLoops] Missing RWProbeOp support, crashes #6820
Comments
dtzSiFive
added
bug
Something isn't working
FIRRTL
Involving the `firrtl` dialect
labels
Mar 13, 2024
I'll PR a workaround to unbreak RWProbe, assigned to you @prithayan since you're familiar and I believe already working on it. LMK if have any questions or can help 👍. (RWProbeOp resolution may be lacking re:ergonomics for this usage presently) |
dtzSiFive
added a commit
to dtzSiFive/circt
that referenced
this issue
Mar 13, 2024
Unbreak this, at cost of missing loops through force, while the better fix is being worked out. cc llvm#6820.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Example FIRRTL:
--parse-only
:Crashes (or assert failure) when trying to resolve the force.
Flagged in review: #5647 (comment) .
The text was updated successfully, but these errors were encountered: