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 support for the blif construct .subckt #21

Open
SebastianBoe opened this issue Dec 20, 2015 · 3 comments
Open

Add support for the blif construct .subckt #21

SebastianBoe opened this issue Dec 20, 2015 · 3 comments

Comments

@SebastianBoe
Copy link

Hi,

from what I see in the blif parser, it appears that the .subckt
construct is not supported yet.

I would like this supported because Yosys generated .subckt
commands when the -noflatten option was used.

Symptom:
top.blif:42: fatal error: unknown directive

Workaround:
Don't use subckt. (For the yosys flow, don't use the -noflatten option)

PS: Thanks for creating this project. Since there exists a workaround, it might make sense
to close this as won't fix.

@cseed
Copy link
Contributor

cseed commented Dec 25, 2015

This would be reasonable to add. It might also be useful for non-yosys tools that want to generate .blif files directly. I'm going to leave it open as an enhancement. Thanks for the suggestion!

@porglezomp
Copy link

I ran into this issue while working with yosys.
Another workaround that might be helpful to people who aren't happy with avoiding -noflatten for any reason. Instead of writing:

synth_ice40 … -blif <name.blif>

you can flatten your design after synthesis and write it then.

synth_ice40 …
flatten
write_blif -gates -attr -param <name.blif>

@whitequark
Copy link
Member

FYI Arachne is essentially not maintained at this point as nextpnr is a far more capable replacement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants