[InstCombine] less indenting; NFC

llvm-svn: 261652

1 files changed, 32 insertions, 31 deletions

diff --git a/llvm/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp b/llvm/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp
index 4ac740ec408..eb41d38c307 100644
--- a/llvm/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp
+++ b/llvm/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp
@@ -1245,38 +1245,39 @@ static Instruction *matchDeMorgansLaws(BinaryOperator &I,
 
 Instruction *InstCombiner::foldCastedBitwiseLogic(BinaryOperator &I) {
   Value *Op0 = I.getOperand(0), *Op1 = I.getOperand(1);
-  if (CastInst *Op0C = dyn_cast<CastInst>(Op0)) {
-    Value *Op0COp = Op0C->getOperand(0);
-    Type *SrcTy = Op0COp->getType();
-    // fold (and (cast A), (cast B)) -> (cast (and A, B))
-    if (CastInst *Op1C = dyn_cast<CastInst>(Op1)) {
-      if (Op0C->getOpcode() == Op1C->getOpcode() && // same cast kind ?
-          SrcTy == Op1C->getOperand(0)->getType() &&
-          SrcTy->isIntOrIntVectorTy()) {
-        Value *Op1COp = Op1C->getOperand(0);
-
-        // Only do this if the casts both really cause code to be generated.
-        if (ShouldOptimizeCast(Op0C->getOpcode(), Op0COp, I.getType()) &&
-            ShouldOptimizeCast(Op1C->getOpcode(), Op1COp, I.getType())) {
-          Value *NewOp = Builder->CreateAnd(Op0COp, Op1COp, I.getName());
-          return CastInst::Create(Op0C->getOpcode(), NewOp, I.getType());
-        }
-
-        // If this is and(cast(icmp), cast(icmp)), try to fold this even if the
-        // cast is otherwise not optimizable.  This happens for vector sexts.
-        if (ICmpInst *RHS = dyn_cast<ICmpInst>(Op1COp))
-          if (ICmpInst *LHS = dyn_cast<ICmpInst>(Op0COp))
-            if (Value *Res = FoldAndOfICmps(LHS, RHS))
-              return CastInst::Create(Op0C->getOpcode(), Res, I.getType());
+  CastInst *Op0C = dyn_cast<CastInst>(Op0);
+  CastInst *Op1C = dyn_cast<CastInst>(Op1);
+  if (!Op0C || !Op1C)
+    return nullptr;
 
-        // If this is and(cast(fcmp), cast(fcmp)), try to fold this even if the
-        // cast is otherwise not optimizable.  This happens for vector sexts.
-        if (FCmpInst *RHS = dyn_cast<FCmpInst>(Op1COp))
-          if (FCmpInst *LHS = dyn_cast<FCmpInst>(Op0COp))
-            if (Value *Res = FoldAndOfFCmps(LHS, RHS))
-              return CastInst::Create(Op0C->getOpcode(), Res, I.getType());
-      }
-    }
+  Value *Op0COp = Op0C->getOperand(0);
+  Type *SrcTy = Op0COp->getType();
+  // fold (and (cast A), (cast B)) -> (cast (and A, B))
+  if (Op0C->getOpcode() == Op1C->getOpcode() && // same cast kind ?
+      SrcTy == Op1C->getOperand(0)->getType() &&
+      SrcTy->isIntOrIntVectorTy()) {
+    Value *Op1COp = Op1C->getOperand(0);
+
+    // Only do this if the casts both really cause code to be generated.
+    if (ShouldOptimizeCast(Op0C->getOpcode(), Op0COp, I.getType()) &&
+        ShouldOptimizeCast(Op1C->getOpcode(), Op1COp, I.getType())) {
+      Value *NewOp = Builder->CreateAnd(Op0COp, Op1COp, I.getName());
+      return CastInst::Create(Op0C->getOpcode(), NewOp, I.getType());
+    }
+
+    // If this is and(cast(icmp), cast(icmp)), try to fold this even if the
+    // cast is otherwise not optimizable.  This happens for vector sexts.
+    if (ICmpInst *RHS = dyn_cast<ICmpInst>(Op1COp))
+      if (ICmpInst *LHS = dyn_cast<ICmpInst>(Op0COp))
+        if (Value *Res = FoldAndOfICmps(LHS, RHS))
+          return CastInst::Create(Op0C->getOpcode(), Res, I.getType());
+
+    // If this is and(cast(fcmp), cast(fcmp)), try to fold this even if the
+    // cast is otherwise not optimizable.  This happens for vector sexts.
+    if (FCmpInst *RHS = dyn_cast<FCmpInst>(Op1COp))
+      if (FCmpInst *LHS = dyn_cast<FCmpInst>(Op0COp))
+        if (Value *Res = FoldAndOfFCmps(LHS, RHS))
+          return CastInst::Create(Op0C->getOpcode(), Res, I.getType());
   }
   return nullptr;
 }