-
Notifications
You must be signed in to change notification settings - Fork 13.5k
[SDAG] Miscompile at O3 #128309
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
Comments
Reduced:
|
@llvm/issue-subscribers-backend-x86 Author: Yingwei Zheng (dtcxzyw)
Reproducer:
```
target datalayout = "e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-i128:128-f80:128-n8:16:32:64-S128"
target triple = "x86_64-unknown-linux-gnu"
@b = dso_local local_unnamed_addr global i64 4073709551615, align 8 define i32 @main() { declare noundef i32 @printf(ptr noundef readonly captures(none), ...)
> ./bin/lli test.ll
|
Caused by #126448. |
…lAdd` (#128353) A counterexample for original implementation: https://alive2.llvm.org/ce/z/7ieYLg This patch uses zext instead of anyext to fix the original issue. BTW, we should keep low `64 - shamt` bits instead of `shamt - 32`: https://alive2.llvm.org/ce/z/ruQP_Z Some codes are simplified to avoid confusion. Proof: https://alive2.llvm.org/ce/z/z_jdHD Closes #128309.
…ei64TruncSrlAdd` (#128353) A counterexample for original implementation: https://alive2.llvm.org/ce/z/7ieYLg This patch uses zext instead of anyext to fix the original issue. BTW, we should keep low `64 - shamt` bits instead of `shamt - 32`: https://alive2.llvm.org/ce/z/ruQP_Z Some codes are simplified to avoid confusion. Proof: https://alive2.llvm.org/ce/z/z_jdHD Closes llvm/llvm-project#128309.
Reproducer:
The text was updated successfully, but these errors were encountered: