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

Support Verilator 5 #75

Open
RickLuiken opened this issue Dec 21, 2023 · 2 comments
Open

Support Verilator 5 #75

RickLuiken opened this issue Dec 21, 2023 · 2 comments

Comments

@RickLuiken
Copy link

Following from the discussion in the fork from KU Leuven. I would like to have support for Verilator 5. Verilator 5 has better performance, support for timing constructs and support for assertions. It would also allow projects using the snitch cluster to use verilator 5. I think in general it would be useful to support more recent versions of tools, as this makes the upgrading process later down the line a lot easier (make small steps instead of big ones).

Upgrading to verilator 5 would involve the following steps:

  • Add the -no-timing flag to the current tests, which disables the new timing features in verilator 5
  • Add verilated_threads.o to the verilator build targets OR compile both the systemverilog and cpp sources using verilator, so verilator can manage which verilator files should be included

If you have any questions or comments, please let me know!

@colluca
Copy link
Collaborator

colluca commented Dec 22, 2023

Dear @RickLuiken,

If you could provide a PR for this feature targeting Verilator 5.006, it would be well accepted and we would be willing to review and merge it.

Unfortunately we do not have enough resources to implement this feature ourselves at the moment.

@RickLuiken
Copy link
Author

Great, I will provide the PR after the christmas break!

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

No branches or pull requests

2 participants