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

Icarus verilog v11 #344

Closed
quark17 opened this issue Mar 30, 2021 · 0 comments · Fixed by #560
Closed

Icarus verilog v11 #344

quark17 opened this issue Mar 30, 2021 · 0 comments · Fixed by #560
Labels
good first issue Good for newcomers testsuite This is an issue with the testsuite

Comments

@quark17
Copy link
Collaborator

quark17 commented Mar 30, 2021

As B-Lang-org/bsc-testsuite#16 (and related PR B-Lang-org/bsc-testsuite#17) show, the way that BSC invokes iverilog for designs with imported foreign functions leads to this warning message:

SFT files are deprecated. Please pass the VPI module instead.

I suspect that the bsc_build_vsim_iverilog script in BSC will need to be updated to check the iverilog version and to use different VPI arguments when the version is 11. This should be simple to fix, but it requires figuring out what the new invocation should look like -- so I've marked this as a "good first issue".

In the testsuite, in bsc.options, the exp-file was updated to filter out this deprecation warning (which was otherwise causing a failure when comparing the expected output). This filtering can be removed when the exec script is fixed.

There are also a number of places in the testsuite where we expect verilog tests to fail, for certain versions of iverilog, due to bugs in the VPI support. It would be good to check to whether any of these bugs have been fixed and can be enabled when the iverilog version is 11.

@quark17 quark17 added good first issue Good for newcomers testsuite This is an issue with the testsuite labels Mar 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers testsuite This is an issue with the testsuite
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant