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

Verilog files issue #124

Closed
nayiri-k opened this issue Oct 13, 2021 · 1 comment
Closed

Verilog files issue #124

nayiri-k opened this issue Oct 13, 2021 · 1 comment

Comments

@nayiri-k
Copy link

I'm trying to use the verilog files generated for the Sky130 SRAMs, but I encounter an error during RTL simulation. It seems the mem signal is used before it is declared (via this declaration: reg [DATA_WIDTH-1:0] mem [0:RAM_DEPTH-1];). Moving this declaration before mem is referenced resolved the error in my simulation.

I am assuming that the open-source simulator used to verify the verilog files doesn't treat undeclared signals as an error, so this issue wasn't detected.

@mguthaus
Copy link
Collaborator

Thanks, this should be fixed in e6a0093 in the dev branch.

Yes, none of the open-source simulators seem to complain about this, so we didn't catch it. Thanks for pointing it out!

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