Skip to content
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

Add proc -nomux to Yosys and migrate to it #479

Closed
whitequark opened this issue Aug 17, 2020 · 1 comment
Closed

Add proc -nomux to Yosys and migrate to it #479

whitequark opened this issue Aug 17, 2020 · 1 comment

Comments

@whitequark
Copy link
Member

@whitequark whitequark commented Aug 17, 2020

The position of Yosys developers (I say with my nMigen hat on) is that using proc_* passes in isolation is likely to break. We currently use them to prepare the netlist for write_verilog, which can't handle the netlist we emit as-is and requires it to be in a rather specific (and somewhat inconvenient) format.

What we do could almost certainly be replaced with proc -nomux (i.e. what the proc wrapper does minus proc_mux), which would be less fragile.

@whitequark
Copy link
Member Author

@whitequark whitequark commented Aug 26, 2020

proc -nomux is now available upstream; we need at least a node version bump to actually use it though.

@whitequark whitequark added this to the 0.3 milestone Nov 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant