This repository has been archived by the owner on Aug 20, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 176
latest FIRRTL generates incorrect code in rocket-chip #705
Comments
(also, how come I didn't get the fancy template for filing my issue? How is that flow supposed to work)? |
Because I forgot to approve and merge #699. Anyway I've narrowed down the bug, fix coming soon. For those interested, if you don't invalidate input ports of a submodule and yet you only connect to them conditionally, it drops most of the conditional connections and picks one (I think the last one). This hasn't come up because Chisel was invalidating everything by default, but with the invalidate API, it doesn't anymore. |
jackkoenig
added a commit
to chipsalliance/chisel
that referenced
this issue
Dec 20, 2017
Fixes #746 Also add test for chipsalliance/firrtl#705
jackkoenig
added a commit
to chipsalliance/chisel
that referenced
this issue
Dec 20, 2017
Fixes #746 Also add test for chipsalliance/firrtl#705
Merged
8 tasks
jackkoenig
added a commit
to chipsalliance/chisel
that referenced
this issue
Dec 20, 2017
…#747) Fixes #746 Also add test for chipsalliance/firrtl#705
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
See chipsalliance/rocket-chip#1160
The text was updated successfully, but these errors were encountered: