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

Reducing iteration count in t/t_merge_cond_blowup causes unrolling error #3451

Closed
gezalore opened this issue May 27, 2022 · 1 comment
Closed
Labels
area: usability Issue involves general usability status: discussion Issue is waiting for discussions to resolve

Comments

@gezalore
Copy link
Member

Reducing N to 16 in t/t_merge_cond_blowup.v here:

localparam int N = 4096;

yields this error:

%Error: t/t_merge_cond_blowup.v:46:9: Loop unrolling took too long; probably this is an infinite loop, or set --unroll-count above 1000000000
   46 |         for (int i = 0 ; i < N ; i = i + 1) begin
      |         ^~~

Surely reducing the iteration count should not cause unrolling to fail, especially given the unrolling limit is set to 1e9.

@gezalore gezalore added area: usability Issue involves general usability status: discussion Issue is waiting for discussions to resolve type: bug labels May 27, 2022
gezalore added a commit to gezalore/verilator that referenced this issue Aug 3, 2022
@gezalore
Copy link
Member Author

gezalore commented Aug 3, 2022

Fixed with f9f66d7

@gezalore gezalore closed this as completed Aug 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: usability Issue involves general usability status: discussion Issue is waiting for discussions to resolve
Projects
None yet
Development

No branches or pull requests

1 participant