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

failure to create header files depending on how parameters are constructed #876

Closed
veripoolbot opened this issue Jan 25, 2015 · 2 comments
Closed
Labels
resolution: abandoned Closed; not enough information or otherwise never finished

Comments

@veripoolbot
Copy link
Contributor


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.

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

@veripoolbot
Copy link
Contributor Author


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.

@veripoolbot
Copy link
Contributor Author


Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2015-09-19T01:01:10Z


Please reopen if the test gets updated, thanks

@veripoolbot veripoolbot added the resolution: abandoned Closed; not enough information or otherwise never finished label Dec 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
resolution: abandoned Closed; not enough information or otherwise never finished
Projects
None yet
Development

No branches or pull requests

1 participant