summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSanjay Patel <spatel@rotateright.com>2016-09-28 13:53:13 +0000
committerSanjay Patel <spatel@rotateright.com>2016-09-28 13:53:13 +0000
commit1b312ad42d1bb898c8f386e0d22f823b5c12f526 (patch)
tree06c331cd97a5fb01aea8b4b2b725b1e321660258
parent47ca99b3726dc6d1f5b0de986848f608741bc5a4 (diff)
downloadbcm5719-llvm-1b312ad42d1bb898c8f386e0d22f823b5c12f526.tar.gz
bcm5719-llvm-1b312ad42d1bb898c8f386e0d22f823b5c12f526.zip
[InstSimplify] allow and-of-icmps folds with vector splat constants
llvm-svn: 282590
-rw-r--r--llvm/lib/Analysis/InstructionSimplify.cpp19
-rw-r--r--llvm/test/Transforms/InstSimplify/AndOrXor.ll66
2 files changed, 75 insertions, 10 deletions
diff --git a/llvm/lib/Analysis/InstructionSimplify.cpp b/llvm/lib/Analysis/InstructionSimplify.cpp
index d42cad92e76..5f38ffe32f7 100644
--- a/llvm/lib/Analysis/InstructionSimplify.cpp
+++ b/llvm/lib/Analysis/InstructionSimplify.cpp
@@ -1518,23 +1518,22 @@ static Value *SimplifyAndOfICmps(ICmpInst *Op0, ICmpInst *Op1) {
return getFalse(ITy);
}
- // FIXME: Use m_APInt to allow vector splat matches.
- ConstantInt *CI1, *CI2;
- if (!match(Op0, m_ICmp(Pred0, m_Add(m_Value(V), m_ConstantInt(CI1)),
- m_ConstantInt(CI2))))
+ // (icmp (add V, C0), C1) & (icmp V, C0)
+ if (!match(Op0, m_ICmp(Pred0, m_Add(m_Value(V), m_APInt(C0)), m_APInt(C1))))
return nullptr;
- if (!match(Op1, m_ICmp(Pred1, m_Specific(V), m_Specific(CI1))))
+ if (!match(Op1, m_ICmp(Pred1, m_Specific(V), m_Value())))
return nullptr;
auto *AddInst = cast<BinaryOperator>(Op0->getOperand(0));
+ if (AddInst->getOperand(1) != Op1->getOperand(1))
+ return nullptr;
+
bool isNSW = AddInst->hasNoSignedWrap();
bool isNUW = AddInst->hasNoUnsignedWrap();
- const APInt &CI1V = CI1->getValue();
- const APInt &CI2V = CI2->getValue();
- const APInt Delta = CI2V - CI1V;
- if (CI1V.isStrictlyPositive()) {
+ const APInt Delta = *C1 - *C0;
+ if (C0->isStrictlyPositive()) {
if (Delta == 2) {
if (Pred0 == ICmpInst::ICMP_ULT && Pred1 == ICmpInst::ICMP_SGT)
return getFalse(ITy);
@@ -1548,7 +1547,7 @@ static Value *SimplifyAndOfICmps(ICmpInst *Op0, ICmpInst *Op1) {
return getFalse(ITy);
}
}
- if (CI1V.getBoolValue() && isNUW) {
+ if (C0->getBoolValue() && isNUW) {
if (Delta == 2)
if (Pred0 == ICmpInst::ICMP_ULT && Pred1 == ICmpInst::ICMP_UGT)
return getFalse(ITy);
diff --git a/llvm/test/Transforms/InstSimplify/AndOrXor.ll b/llvm/test/Transforms/InstSimplify/AndOrXor.ll
index 2442e684246..fb2751e708d 100644
--- a/llvm/test/Transforms/InstSimplify/AndOrXor.ll
+++ b/llvm/test/Transforms/InstSimplify/AndOrXor.ll
@@ -49,6 +49,17 @@ define i1 @and_of_icmps0(i32 %b) {
ret i1 %cmp
}
+define <2 x i1> @and_of_icmps0_vec(<2 x i32> %b) {
+; CHECK-LABEL: @and_of_icmps0_vec(
+; CHECK-NEXT: ret <2 x i1> zeroinitializer
+;
+ %1 = add <2 x i32> %b, <i32 2, i32 2>
+ %2 = icmp ult <2 x i32> %1, <i32 4, i32 4>
+ %cmp3 = icmp sgt <2 x i32> %b, <i32 2, i32 2>
+ %cmp = and <2 x i1> %2, %cmp3
+ ret <2 x i1> %cmp
+}
+
define i1 @and_of_icmps1(i32 %b) {
; CHECK-LABEL: @and_of_icmps1(
; CHECK-NEXT: ret i1 false
@@ -60,6 +71,17 @@ define i1 @and_of_icmps1(i32 %b) {
ret i1 %cmp
}
+define <2 x i1> @and_of_icmps1_vec(<2 x i32> %b) {
+; CHECK-LABEL: @and_of_icmps1_vec(
+; CHECK-NEXT: ret <2 x i1> zeroinitializer
+;
+ %1 = add nsw <2 x i32> %b, <i32 2, i32 2>
+ %2 = icmp slt <2 x i32> %1, <i32 4, i32 4>
+ %cmp3 = icmp sgt <2 x i32> %b, <i32 2, i32 2>
+ %cmp = and <2 x i1> %2, %cmp3
+ ret <2 x i1> %cmp
+}
+
define i1 @and_of_icmps2(i32 %b) {
; CHECK-LABEL: @and_of_icmps2(
; CHECK-NEXT: ret i1 false
@@ -71,6 +93,17 @@ define i1 @and_of_icmps2(i32 %b) {
ret i1 %cmp
}
+define <2 x i1> @and_of_icmps2_vec(<2 x i32> %b) {
+; CHECK-LABEL: @and_of_icmps2_vec(
+; CHECK-NEXT: ret <2 x i1> zeroinitializer
+;
+ %1 = add <2 x i32> %b, <i32 2, i32 2>
+ %2 = icmp ule <2 x i32> %1, <i32 3, i32 3>
+ %cmp3 = icmp sgt <2 x i32> %b, <i32 2, i32 2>
+ %cmp = and <2 x i1> %2, %cmp3
+ ret <2 x i1> %cmp
+}
+
define i1 @and_of_icmps3(i32 %b) {
; CHECK-LABEL: @and_of_icmps3(
; CHECK-NEXT: ret i1 false
@@ -82,6 +115,17 @@ define i1 @and_of_icmps3(i32 %b) {
ret i1 %cmp
}
+define <2 x i1> @and_of_icmps3_vec(<2 x i32> %b) {
+; CHECK-LABEL: @and_of_icmps3_vec(
+; CHECK-NEXT: ret <2 x i1> zeroinitializer
+;
+ %1 = add nsw <2 x i32> %b, <i32 2, i32 2>
+ %2 = icmp sle <2 x i32> %1, <i32 3, i32 3>
+ %cmp3 = icmp sgt <2 x i32> %b, <i32 2, i32 2>
+ %cmp = and <2 x i1> %2, %cmp3
+ ret <2 x i1> %cmp
+}
+
define i1 @and_of_icmps4(i32 %b) {
; CHECK-LABEL: @and_of_icmps4(
; CHECK-NEXT: ret i1 false
@@ -93,6 +137,17 @@ define i1 @and_of_icmps4(i32 %b) {
ret i1 %cmp
}
+define <2 x i1> @and_of_icmps4_vec(<2 x i32> %b) {
+; CHECK-LABEL: @and_of_icmps4_vec(
+; CHECK-NEXT: ret <2 x i1> zeroinitializer
+;
+ %1 = add nuw <2 x i32> %b, <i32 2, i32 2>
+ %2 = icmp ult <2 x i32> %1, <i32 4, i32 4>
+ %cmp3 = icmp ugt <2 x i32> %b, <i32 2, i32 2>
+ %cmp = and <2 x i1> %2, %cmp3
+ ret <2 x i1> %cmp
+}
+
define i1 @and_of_icmps5(i32 %b) {
; CHECK-LABEL: @and_of_icmps5(
; CHECK-NEXT: ret i1 false
@@ -104,6 +159,17 @@ define i1 @and_of_icmps5(i32 %b) {
ret i1 %cmp
}
+define <2 x i1> @and_of_icmps5_vec(<2 x i32> %b) {
+; CHECK-LABEL: @and_of_icmps5_vec(
+; CHECK-NEXT: ret <2 x i1> zeroinitializer
+;
+ %1 = add nuw <2 x i32> %b, <i32 2, i32 2>
+ %2 = icmp ule <2 x i32> %1, <i32 3, i32 3>
+ %cmp3 = icmp ugt <2 x i32> %b, <i32 2, i32 2>
+ %cmp = and <2 x i1> %2, %cmp3
+ ret <2 x i1> %cmp
+}
+
define i1 @or_of_icmps0(i32 %b) {
; CHECK-LABEL: @or_of_icmps0(
; CHECK-NEXT: ret i1 true
OpenPOWER on IntegriCloud