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

Verilator path forward #3896

Open
ktbarrett opened this issue May 16, 2024 · 0 comments
Open

Verilator path forward #3896

ktbarrett opened this issue May 16, 2024 · 0 comments
Labels
category:simulators:verilator Verilator type:bug a bug in existing functionality

Comments

@ktbarrett
Copy link
Member

The in development version of Verilator (v5.025) introduces support for vpiInertialDelay, however it is up to the user to decide when to apply those writes. This means we are going to have to make changes to verilator.cpp. While I see those changes coming in the near future, cocotb will be broken with verilator until some fix is released.

If 1.9 is released soon, I suggest we introduce an ifdef for Verilator in out VPI implementation to use vpiNoDelay instead, which was the old behavior. If 1.9 is released soon there will only be a small gap where there is no support. This will also allow us to continue to support older version of Verilator that ignore the delay flag.

For 2.0 we will need to update the verilator.cpp.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:simulators:verilator Verilator type:bug a bug in existing functionality
Projects
None yet
Development

No branches or pull requests

1 participant