[PATCH] D134892: [X86] Avoid miscompile in combineOr (X86ISelLowering.cpp)
Bjorn Pettersson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 29 12:26:09 PDT 2022
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG0513b0305acd: [X86] Avoid miscompile in combineOr (X86ISelLowering.cpp) (authored by bjope).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134892/new/
https://reviews.llvm.org/D134892
Files:
llvm/lib/Target/X86/X86ISelLowering.cpp
llvm/test/CodeGen/X86/or-lea.ll
Index: llvm/test/CodeGen/X86/or-lea.ll
===================================================================
--- llvm/test/CodeGen/X86/or-lea.ll
+++ llvm/test/CodeGen/X86/or-lea.ll
@@ -811,10 +811,12 @@
;
; X64-LABEL: or_large_constant:
; X64: # %bb.0: # %entry
-; X64-NEXT: xorl %eax, %eax
+; X64-NEXT: xorl %ecx, %ecx
; X64-NEXT: cmpq $2, %rdi
-; X64-NEXT: setl %al
-; X64-NEXT: leaq -1(%rax,%rax), %rax
+; X64-NEXT: setge %cl
+; X64-NEXT: negq %rcx
+; X64-NEXT: movabsq $549755813889, %rax # imm = 0x8000000001
+; X64-NEXT: orq %rcx, %rax
; X64-NEXT: retq
entry:
%cmp = icmp sgt i64 %x, 1
Index: llvm/lib/Target/X86/X86ISelLowering.cpp
===================================================================
--- llvm/lib/Target/X86/X86ISelLowering.cpp
+++ llvm/lib/Target/X86/X86ISelLowering.cpp
@@ -48592,7 +48592,7 @@
if (Cond.getOpcode() == X86ISD::SETCC && Cond.hasOneUse()) {
if (auto *CN = dyn_cast<ConstantSDNode>(N1)) {
- unsigned Val = CN->getZExtValue();
+ uint64_t Val = CN->getZExtValue();
if (Val == 1 || Val == 2 || Val == 3 || Val == 4 || Val == 7 || Val == 8) {
X86::CondCode CCode = (X86::CondCode)Cond.getConstantOperandVal(0);
CCode = X86::GetOppositeBranchCondition(CCode);
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D134892.463997.patch
Type: text/x-patch
Size: 1295 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220929/a4796800/attachment.bin>
More information about the llvm-commits
mailing list