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

Compiler bug when generating SRAM blocks using a for statement #1470

Open
joonho3020 opened this issue Jun 11, 2024 · 3 comments
Open

Compiler bug when generating SRAM blocks using a for statement #1470

joonho3020 opened this issue Jun 11, 2024 · 3 comments
Labels
question Further information is requested

Comments

@joonho3020
Copy link

Describe the bug
A clear and concise description of what the bug is.

Compiler errors out when trying to run functional simulation for a fairly complex block.

To Reproduce
Steps to reproduce the behavior:

  • Follow the XLS setup instructions for Ubuntu 2024
  • Git clone my XLS fork
  • Run bazel run -- //xls/modules/snappy:snappy_decomp_test --alsologtostderr --compare=none

Expected behavior
A clear and concise description of what you expected to happen.

Simulation runs. However I get a compile error here : error source

If I hand unroll the above loop, I can run simulations without errors, so I'm assuming that this is a frontend compiler bug.

Screenshots
If applicable, add screenshots to help explain your problem.

https://github.com/joonho3020/xls/blob/1ab579996fabd05a39b35818ce71ad2e56b21f8f/xls/modules/snappy/command_executer.x#L103

Environment (this can be helpful for troubleshooting):

  • OS: [e.g. Ubuntu]
  • Versions [e.g. 22.04]

Additional context
Add any other context about the problem here.

@hzeller
Copy link
Member

hzeller commented Jun 11, 2024

Same comment like #1469 (comment) applies here.

Can you provide a reproducible example that runs against the head of the main repository not your personal fork ?

@joonho3020
Copy link
Author

Same as #1469 : would it make is easier if I PR the module in the XLS repo?

@proppy
Copy link
Member

proppy commented Jun 13, 2024

@joonho3020 can you try to use https://bit.ly/xls-playground to reproduce the error?

@proppy proppy added the question Further information is requested label Jun 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants