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

Does synlig plugin attempt to load write_ilang ? #2677

Closed
hzeller opened this issue Nov 26, 2024 · 2 comments · Fixed by #2685
Closed

Does synlig plugin attempt to load write_ilang ? #2677

hzeller opened this issue Nov 26, 2024 · 2 comments · Fixed by #2685

Comments

@hzeller
Copy link
Collaborator

hzeller commented Nov 26, 2024

When attempting to load the synlig plugin, yosys errors out complaining that write_ilang pass already exists, as if the synlig plugin recusively attempts to load that other plugin ?

yosys> plugin -i path/to/systemverilog.so
Warning: Using synlig as yosys plugin is deprecated. It is recommended to build synlig as standalone binary.
ERROR: Unable to register pass 'write_ilang', pass already exists!
@kamilrakoczy
Copy link
Collaborator

Yosys recently dropped read/write_ilang alias: YosysHQ/yosys#4704

We added this alias back on synlig side when we were updating yosys version: #2668.

In #2685 we added check if this pass is already registered and only register this alias if it isn't.

We encourage users to use synlig as as standalone binary to make sure we don't see such issues.

@hzeller
Copy link
Collaborator Author

hzeller commented Dec 3, 2024

Thanks.
Right now I need the plug-in feature as I am maintaining the NixOS package, and having a smoothly working plugin that works with an existing Yosys is preferrable from the packaging point-of-view than pulling in a vendored yosys.

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

Successfully merging a pull request may close this issue.

2 participants