[llvm] ca51529 - [SystemZ] Extend combineGET_CCMASK() to handle a truncated SELECT_CCMASK.
Jonas Paulsson via llvm-commits
llvm-commits at lists.llvm.org
Wed Nov 23 06:54:23 PST 2022
Author: Jonas Paulsson
Date: 2022-11-23T09:53:07-05:00
New Revision: ca515294879652a46c0e76aaa66f8e415e308b8d
URL: https://github.com/llvm/llvm-project/commit/ca515294879652a46c0e76aaa66f8e415e308b8d
DIFF: https://github.com/llvm/llvm-project/commit/ca515294879652a46c0e76aaa66f8e415e308b8d.diff
LOG: [SystemZ] Extend combineGET_CCMASK() to handle a truncated SELECT_CCMASK.
In cases where the SELECT_CCMASK has an additional user of the carry, a
truncated SELECT_CCMASK may result as the input to the GET_CCMASK, which need
to be recognized.
Fixes https://github.com/llvm/llvm-project/issues/59054
Reviewed By: Ulrich Weigand
Differential Revision: https://reviews.llvm.org/D138324
Added:
llvm/test/CodeGen/SystemZ/int-uadd-multiple-carry-users.ll
Modified:
llvm/lib/Target/SystemZ/SystemZISelLowering.cpp
Removed:
################################################################################
diff --git a/llvm/lib/Target/SystemZ/SystemZISelLowering.cpp b/llvm/lib/Target/SystemZ/SystemZISelLowering.cpp
index 95d87197b0f4d..01f47e63ad92c 100644
--- a/llvm/lib/Target/SystemZ/SystemZISelLowering.cpp
+++ b/llvm/lib/Target/SystemZ/SystemZISelLowering.cpp
@@ -7038,6 +7038,8 @@ SDValue SystemZTargetLowering::combineGET_CCMASK(
int CCMaskVal = CCMask->getZExtValue();
SDValue Select = N->getOperand(0);
+ if (Select->getOpcode() == ISD::TRUNCATE)
+ Select = Select->getOperand(0);
if (Select->getOpcode() != SystemZISD::SELECT_CCMASK)
return SDValue();
@@ -7052,9 +7054,9 @@ SDValue SystemZTargetLowering::combineGET_CCMASK(
auto *FalseVal = dyn_cast<ConstantSDNode>(Select->getOperand(1));
if (!TrueVal || !FalseVal)
return SDValue();
- if (TrueVal->getZExtValue() != 0 && FalseVal->getZExtValue() == 0)
+ if (TrueVal->getZExtValue() == 1 && FalseVal->getZExtValue() == 0)
;
- else if (TrueVal->getZExtValue() == 0 && FalseVal->getZExtValue() != 0)
+ else if (TrueVal->getZExtValue() == 0 && FalseVal->getZExtValue() == 1)
SelectCCMaskVal ^= SelectCCValidVal;
else
return SDValue();
diff --git a/llvm/test/CodeGen/SystemZ/int-uadd-multiple-carry-users.ll b/llvm/test/CodeGen/SystemZ/int-uadd-multiple-carry-users.ll
new file mode 100644
index 0000000000000..6d57dcfefd423
--- /dev/null
+++ b/llvm/test/CodeGen/SystemZ/int-uadd-multiple-carry-users.ll
@@ -0,0 +1,23 @@
+; RUN: llc < %s -mtriple=s390x-linux-gnu -start-after=codegenprepare | FileCheck %s
+;
+; Test that compilation succeeds with multiple users of the carry resulting
+; in a truncation of the SELECT_CCMASK used by GET_CCMASK.
+
+define void @fun() {
+; CHECK-LABEL: fun:
+bb:
+ %0 = call { i64, i1 } @llvm.uadd.with.overflow.i64(i64 poison, i64 poison)
+ %ov = extractvalue { i64, i1 } %0, 1
+ %i2 = sext i1 %ov to i64
+ %i3 = select i1 %ov, i64 9223372036854775807, i64 -1
+ %i4 = sub nsw i64 %i3, %i2
+ %i5 = and i64 %i4, %i2
+ %i6 = icmp slt i64 %i5, 0
+ %i7 = xor i1 %ov, true
+ %i8 = select i1 %i6, i1 %i7, i1 false
+ %i9 = sext i1 %i8 to i16
+ store i16 %i9, ptr poison, align 2
+ unreachable
+}
+
+declare { i64, i1 } @llvm.uadd.with.overflow.i64(i64, i64)
More information about the llvm-commits
mailing list