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

Yosys verilog reader enhancement request #1530

Open
shakouri opened this issue Nov 29, 2022 · 4 comments
Open

Yosys verilog reader enhancement request #1530

shakouri opened this issue Nov 29, 2022 · 4 comments
Labels
enhancement New feature or request OpenLane 2 Scheduled for next gen OpenLane Yosys This issue is related to yosys

Comments

@shakouri
Copy link

Description

The yosys verilog reader is not able to parse verilog code that other synthesis tools are able to parse.

  1. Allow parameter passing w/o use of parantheses ('-' denotes original RTL, and '+' denotes changes to make yosys read it in):

yosys_case1

  1. Allow use of ? during async portion of always block ('-' denotes original RTL, and '+' denotes changes to make yosys read it in):

yosys_case2

yosys_case3

Proposal

The yosys verilog reader will need to be enhanced.

@mithro
Copy link

mithro commented Nov 29, 2022

This is just an FYI;

There are a couple of potential options for improve language support in Yosys;

I have a diagram of the last two @ https://j.mp/sv-flow-diagram

@shakouri
Copy link
Author

I tried sv2v, but the tool is not also able to read it either. This is error that it gives:
Parse error: unexpected token '4' (Lit_number)

I am getting the following error when I try to use system verilog plugin:
yosys> plugin -i systemverilog
ERROR: Can't load module `./systemverilog': dlopen(/usr/local/bin/../share/yosys/plugins/systemverilog.so, 5): no suitable image found. Did find:
/usr/local/bin/../share/yosys/plugins/systemverilog.so: unknown file type, first eight bytes: 0x7F 0x45 0x4C 0x46 0x02 0x01 0x01 0x03
/usr/local/Cellar/yosys/0.23/share/yosys/plugins/systemverilog.so: unknown file type, first eight bytes: 0x7F 0x45 0x4C 0x46 0x02 0x01 0x01 0x03

@mithro
Copy link

mithro commented Nov 29, 2022

@shakouri - Please report the systemverilog issues to the issue tracker @ https://github.com/chipsalliance/yosys-f4pga-plugins/issues

@donn donn added enhancement New feature or request Yosys This issue is related to yosys Flow Script labels Nov 30, 2022
@shakouri
Copy link
Author

Ok. Here is the link:
chipsalliance/yosys-f4pga-plugins#412

@donn donn removed the Flow Script label Dec 28, 2022
@kareefardi kareefardi added the OpenLane 2 Scheduled for next gen OpenLane label Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request OpenLane 2 Scheduled for next gen OpenLane Yosys This issue is related to yosys
Projects
None yet
Development

No branches or pull requests

4 participants