simplify code.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@59390 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Transforms/Scalar/InstructionCombining.cpp b/lib/Transforms/Scalar/InstructionCombining.cpp
index c2df3a5b24350861853f605755390233515e33df..7e7372779422e3c488de1461fc39b01c259fdf22 100644 (file)
--- a/lib/Transforms/Scalar/InstructionCombining.cpp
+++ b/lib/Transforms/Scalar/InstructionCombining.cpp
@@ -4210,30 +4210,25 @@ Instruction *InstCombiner::MatchBSwap(BinaryOperator &I) {
 /// we can simplify this expression to "cond ? C : D or B".
 static Instruction *MatchSelectFromAndOr(Value *A, Value *B,
                                          Value *C, Value *D) {
-  // If A is not a select of constants, this can't match.
+  // If A is not a select of -1/0, this cannot match.
   Value *Cond = 0, *Cond2 = 0;
-  ConstantInt *C1, *C2;
-  if (!match(A, m_Select(m_Value(Cond), m_ConstantInt(C1), m_ConstantInt(C2))))
+  if (!match(A, m_Select(m_Value(Cond), m_ConstantInt(-1), m_ConstantInt(0))))
     return 0;
 
 #define SELECT_MATCH(Val, I1, I2) \
   m_Select(m_Value(Val), m_ConstantInt(I1), m_ConstantInt(I2))
   
-  // Handle "cond ? -1 : 0".
-  if (C1->isAllOnesValue() && C2->isZero()) {
-    // ((cond?-1:0)&C) | (B&(cond?0:-1)) -> cond ? C : B.
-    if (match(D, SELECT_MATCH(Cond2, 0, -1)) && Cond2 == Cond)
-      return SelectInst::Create(Cond, C, B);
-    if (match(D, m_Not(SELECT_MATCH(Cond2, -1, 0))) && Cond2 == Cond)
-      return SelectInst::Create(Cond, C, B);
-    // ((cond?-1:0)&C) | ((cond?0:-1)&D) -> cond ? C : D.
-    if (match(B, SELECT_MATCH(Cond2, 0, -1)) && Cond2 == Cond)
-      return SelectInst::Create(Cond, C, D);
-    if (match(B, m_Not(SELECT_MATCH(Cond2, -1, 0))) && Cond2 == Cond)
-      return SelectInst::Create(Cond, C, D);
-  }
+  // ((cond?-1:0)&C) | (B&(cond?0:-1)) -> cond ? C : B.
+  if (match(D, SELECT_MATCH(Cond2, 0, -1)) && Cond2 == Cond)
+    return SelectInst::Create(Cond, C, B);
+  if (match(D, m_Not(SELECT_MATCH(Cond2, -1, 0))) && Cond2 == Cond)
+    return SelectInst::Create(Cond, C, B);
+  // ((cond?-1:0)&C) | ((cond?0:-1)&D) -> cond ? C : D.
+  if (match(B, SELECT_MATCH(Cond2, 0, -1)) && Cond2 == Cond)
+    return SelectInst::Create(Cond, C, D);
+  if (match(B, m_Not(SELECT_MATCH(Cond2, -1, 0))) && Cond2 == Cond)
+    return SelectInst::Create(Cond, C, D);
 #undef SELECT_MATCH
-
   return 0;
 }