You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Verilator fails to build header files if parameters are defined in a few ways, I will describe how to reproduce the issue on a project of mine on a branch with removed extra dependencies.
There seem to be no errors warning or crashes reported by Verilator, just a missing header file.
git clone https://github.com/jeras/rp_8bit.git rp_8bit_verilatorbug
cd rp_8bit_verilatorbug
git checkout fbe357363d9ce355244e19f24136250a5a9e80d2
cd sim/
make verilator
To compare the failing code to a working version checkout the next commit:
git checkout verilator_parameterization_bug
make verilator
For parameters defined as "parameter int unsigned" it seems to be enough to redefine them omitting the type, the same does not seem to be enough for the HEX constant. Using typed parameter definitions inside RTL does not seem to be an issue.
Regards,
Iztok Jeras
The text was updated successfully, but these errors were encountered:
Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2015-01-25T21:43:53Z
I get all sorts of compile errors with your example which aren't related to this bug. Can you please make a standalone test in verilator test format? Thanks.
Author Name: Iztok Jeras (@jeras)
Original Redmine Issue: 876 from https://www.veripool.org
Original Date: 2015-01-25
Hi,
Verilator fails to build header files if parameters are defined in a few ways, I will describe how to reproduce the issue on a project of mine on a branch with removed extra dependencies.
https://github.com/jeras/rp_8bit/tree/verilator_parameterization_bug
There seem to be no errors warning or crashes reported by Verilator, just a missing header file.
To compare the failing code to a working version checkout the next commit:
For parameters defined as "parameter int unsigned" it seems to be enough to redefine them omitting the type, the same does not seem to be enough for the HEX constant. Using typed parameter definitions inside RTL does not seem to be an issue.
Regards,
Iztok Jeras
The text was updated successfully, but these errors were encountered: