-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
cfg_simplify!
error
#52058
Labels
compiler:optimizer
Optimization passes (mostly in base/compiler/ssair/)
Comments
aviatesk
added
the
compiler:optimizer
Optimization passes (mostly in base/compiler/ssair/)
label
Nov 7, 2023
topolarity
added a commit
to topolarity/julia
that referenced
this issue
Nov 9, 2023
This code was using the sentinel value -1 as a special marker in addition to the negative BB indices. That ambiguity was causing `cfg_simplify!` to fail on functions that merge any BB into the first basic block. This change is to use `typemin(Int)` as a marker instead. Fixes JuliaLang#52058
topolarity
added a commit
to topolarity/julia
that referenced
this issue
Nov 10, 2023
This code was using the sentinel value -1 as a special marker in addition to the negative BB indices. That ambiguity was causing `cfg_simplify!` to fail on functions that merge any BB into the first basic block. This change is to use `typemin(Int)` as a marker instead. Fixes JuliaLang#52058
aviatesk
pushed a commit
that referenced
this issue
Nov 11, 2023
This code was using the sentinel value -1 as a special marker in addition to the negative BB indices. That ambiguity was causing `cfg_simplify!` to fail on functions that merge any BB into the first basic block. This change is to use `typemin(Int)` as a marker instead. Fixes #52058
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
MRE:
This issue particularly blocks #51960, which introduces actual use of
cfg_simplify!
into the base compilation pipeline. To echo what Keno mentioned in our previous Slack chat:The text was updated successfully, but these errors were encountered: