-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[erlc] internal error in pass beam_kernel_to_ssa #6572
Comments
bjorng
added a commit
to bjorng/otp
that referenced
this issue
Dec 19, 2022
The `sys_core_fold` does a fixpoint iteration, limited to at most 20 iterations. One of the optimizations (coalescing two `let` constructs) would temporarily generate unsafe code, which would eventually be corrected, **unless** the 20 iteration limit was reached. Closes erlang#6572
bjorng
added a commit
to bjorng/otp
that referenced
this issue
Dec 19, 2022
The `sys_core_fold` does a fixpoint iteration, limited to at most 20 iterations. One of the optimizations (coalescing two `let` constructs) would temporarily generate unsafe code, which would eventually be corrected, **unless** the 20 iteration limit was reached. Closes erlang#6572
The reason you could not minimize the test case is that the fixpoint iteration limit needed be reached for the bug to appear. |
Thank you for the explanation (and for the fix!), I was surprised by the size of the testcase but it makes sense now. |
bjorng
added a commit
to bjorng/otp
that referenced
this issue
Dec 19, 2022
…master/primary_preloaded * bjorn/compiler/sys_core_fold/erlangGH-6572: Eliminate crash in beam_kernel_to_ssa
bjorng
added a commit
that referenced
this issue
Dec 20, 2022
…6572 Eliminate crash in beam_kernel_to_ssa
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
On master (with and without cherry-picking #6559), the following testcase:
causes the following erlc error:
I've been unable to minimize the testcase any further, all parts of it seem required to reproduce this bug.
The text was updated successfully, but these errors were encountered: