-
Notifications
You must be signed in to change notification settings - Fork 11.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
[GVN] GVNPass forgets to remove poison generating flags #82884
Labels
Comments
dtcxzyw
added
the
needs-reduction
Large reproducer that should be reduced into a simpler form
label
Feb 25, 2024
dtcxzyw
added
llvm:GVN
GVN and NewGVN stages (Global value numbering)
and removed
llvm:instcombine
needs-reduction
Large reproducer that should be reduced into a simpler form
labels
Feb 25, 2024
dtcxzyw
changed the title
Miscompilation in
[GVN] GVNPass forgets to remove poison generating flags
Feb 25, 2024
InstCombinePass
Reproducer: https://godbolt.org/z/xs1rhnMvb I will post a fix later. |
dtcxzyw
added a commit
that referenced
this issue
Feb 26, 2024
… intrinsic with a overflowing binary operator (#82935) Alive2: https://alive2.llvm.org/ce/z/gyL7mn Fixes #82884.
llvmbot
pushed a commit
to llvmbot/llvm-project
that referenced
this issue
Feb 26, 2024
… intrinsic with a overflowing binary operator (llvm#82935) Alive2: https://alive2.llvm.org/ce/z/gyL7mn Fixes llvm#82884. (cherry picked from commit 892b4be)
EugeneZelenko
added
llvm:transforms
and removed
llvm:GVN
GVN and NewGVN stages (Global value numbering)
labels
Feb 26, 2024
llvmbot
pushed a commit
to llvmbot/llvm-project
that referenced
this issue
Feb 26, 2024
… intrinsic with a overflowing binary operator (llvm#82935) Alive2: https://alive2.llvm.org/ce/z/gyL7mn Fixes llvm#82884. (cherry picked from commit 892b4be)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Right:
Wrong:
Bisects down to
Version:
From Rust custom MIR
The text was updated successfully, but these errors were encountered: