More styalistic changes. No functionality change.
authorBill Wendling <isanbard@gmail.com>
Tue, 2 Dec 2008 06:18:11 +0000 (06:18 +0000)
committerBill Wendling <isanbard@gmail.com>
Tue, 2 Dec 2008 06:18:11 +0000 (06:18 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@60401 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Transforms/Scalar/InstructionCombining.cpp

index 34c5fb3375d1a5f5311f781a6abbd595fb37fddb..db2c997d3fbe263b9073efffa7869c88c084766d 100644 (file)
@@ -4438,22 +4438,19 @@ Instruction *InstCombiner::FoldOrWithConstants(BinaryOperator &I, Value *Op,
   if (!CI1) return 0;
 
   Value *V1 = 0, *C2 = 0;
-  if (match(Op, m_And(m_Value(V1), m_Value(C2)))) {
-    ConstantInt *CI2 = dyn_cast<ConstantInt>(C2);
-    if (!CI2) return 0;
+  if (!match(Op, m_And(m_Value(V1), m_ConstantInt(C2)))) return 0;
 
-    APInt Xor = CI1->getValue() ^ CI2->getValue();
-    if (Xor.isAllOnesValue()) {
-      if (V1 == B) {
-        Instruction *NewOp =
-          InsertNewInstBefore(BinaryOperator::CreateAnd(A, CI1), I);
-        return BinaryOperator::CreateOr(NewOp, B);
-      } else if (V1 == A) {
-        Instruction *NewOp =
-          InsertNewInstBefore(BinaryOperator::CreateAnd(B, CI1), I);
-        return BinaryOperator::CreateOr(NewOp, A);
-      }
-    }
+  APInt Xor = CI1->getValue() ^ CI2->getValue();
+  if (!Xor.isAllOnesValue()) return 0;
+
+  if (V1 == B) {
+    Instruction *NewOp =
+      InsertNewInstBefore(BinaryOperator::CreateAnd(A, CI1), I);
+    return BinaryOperator::CreateOr(NewOp, B);
+  } else if (V1 == A) {
+    Instruction *NewOp =
+      InsertNewInstBefore(BinaryOperator::CreateAnd(B, CI1), I);
+    return BinaryOperator::CreateOr(NewOp, A);
   }
 
   return 0;