[InstCombine][NFC] Add a ``break;`` statement.
authorSanjoy Das <sanjoy@playingwithpointers.com>
Fri, 5 Jun 2015 18:04:46 +0000 (18:04 +0000)
committerSanjoy Das <sanjoy@playingwithpointers.com>
Fri, 5 Jun 2015 18:04:46 +0000 (18:04 +0000)
This change is NFC because both the ``break;`` and the fall through end
up returning immediately. However, this helps clarify intent and also
ensures correctness in case more ``case`` blocks are added later.

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

lib/Transforms/InstCombine/InstCombineCompares.cpp

index ab0d1b10c2455f5cf11dcfa30b6b4176dbb84189..f53eeef1dae60cb0380e8376de948c060eeab56f 100644 (file)
@@ -2195,6 +2195,7 @@ bool InstCombiner::OptimizeOverflowCheck(OverflowCheckFlavor OCF, Value *LHS,
       if (WillNotOverflowSignedMul(LHS, RHS, OrigI))
         return SetResult(Builder->CreateNSWMul(LHS, RHS), Builder->getFalse(),
                          true);
+    break;
   }
 
   return false;