diff options
| author | Marcello Maggioni <hayarms@gmail.com> | 2017-01-14 07:28:47 +0000 |
|---|---|---|
| committer | Marcello Maggioni <hayarms@gmail.com> | 2017-01-14 07:28:47 +0000 |
| commit | 0616b5ff5c58aadfdf208b9350448f4ab0e8281f (patch) | |
| tree | 3f03b238b33b11d28c8fc94c7eec2e38d6ead8fc /llvm/lib/Analysis | |
| parent | 67e94fb435de3a89346481b39faa8c5af3c4e829 (diff) | |
| download | bcm5719-llvm-0616b5ff5c58aadfdf208b9350448f4ab0e8281f.tar.gz bcm5719-llvm-0616b5ff5c58aadfdf208b9350448f4ab0e8281f.zip | |
Removing potentially error-prone fallthrough. NFC
This fallthrough if other cases are added between fabs and default
could cause fabs to fall to the next case resulting in a bug.
Better getting rid of it immediately just to be sure.
llvm-svn: 292003
Diffstat (limited to 'llvm/lib/Analysis')
| -rw-r--r-- | llvm/lib/Analysis/InstructionSimplify.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/llvm/lib/Analysis/InstructionSimplify.cpp b/llvm/lib/Analysis/InstructionSimplify.cpp index 796e6e44498..6c731c30c4c 100644 --- a/llvm/lib/Analysis/InstructionSimplify.cpp +++ b/llvm/lib/Analysis/InstructionSimplify.cpp @@ -4267,6 +4267,7 @@ static Value *SimplifyIntrinsic(Function *F, IterTy ArgBegin, IterTy ArgEnd, case Intrinsic::fabs: { if (SignBitMustBeZero(*ArgBegin, Q.TLI)) return *ArgBegin; + return nullptr; } default: return nullptr; |
