diff options
| author | Sanjoy Das <sanjoy@playingwithpointers.com> | 2015-06-05 18:04:46 +0000 |
|---|---|---|
| committer | Sanjoy Das <sanjoy@playingwithpointers.com> | 2015-06-05 18:04:46 +0000 |
| commit | c80dad6f18bbd2a1111df3e6d1fde20661094720 (patch) | |
| tree | d3dbde53ebccb9b4ddcfc11989a1c23712e65574 /llvm/lib | |
| parent | 72cb5e1087ff018ba9eb0a226d41a8cc5bfe6a03 (diff) | |
| download | bcm5719-llvm-c80dad6f18bbd2a1111df3e6d1fde20661094720.tar.gz bcm5719-llvm-c80dad6f18bbd2a1111df3e6d1fde20661094720.zip | |
[InstCombine][NFC] Add a ``break;`` statement.
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.
llvm-svn: 239172
Diffstat (limited to 'llvm/lib')
| -rw-r--r-- | llvm/lib/Transforms/InstCombine/InstCombineCompares.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/llvm/lib/Transforms/InstCombine/InstCombineCompares.cpp b/llvm/lib/Transforms/InstCombine/InstCombineCompares.cpp index ab0d1b10c24..f53eeef1dae 100644 --- a/llvm/lib/Transforms/InstCombine/InstCombineCompares.cpp +++ b/llvm/lib/Transforms/InstCombine/InstCombineCompares.cpp @@ -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; |

