diff options
| author | Richard Trieu <rtrieu@google.com> | 2016-02-18 22:09:30 +0000 |
|---|---|---|
| committer | Richard Trieu <rtrieu@google.com> | 2016-02-18 22:09:30 +0000 |
| commit | 7a08381403b54cd8998f3c922f18b65867e3c07c (patch) | |
| tree | 888b16f3115d38f20de5fefd2d736c173a4fdc76 /llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp | |
| parent | ec3d33274887bc59ff3bfdba90494a63850680cc (diff) | |
| download | llvm-7a08381403b54cd8998f3c922f18b65867e3c07c.zip llvm-7a08381403b54cd8998f3c922f18b65867e3c07c.tar.gz llvm-7a08381403b54cd8998f3c922f18b65867e3c07c.tar.bz2 | |
Remove uses of builtin comma operator.
Cleanup for upcoming Clang warning -Wcomma. No functionality change intended.
llvm-svn: 261270
Diffstat (limited to 'llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp')
| -rw-r--r-- | llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp b/llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp index 6a7e29be..4c3dccb 100644 --- a/llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp +++ b/llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp @@ -374,10 +374,13 @@ Instruction *InstCombiner::visitMul(BinaryOperator &I) { APInt Negative2(I.getType()->getPrimitiveSizeInBits(), (uint64_t)-2, true); Value *BoolCast = nullptr, *OtherOp = nullptr; - if (MaskedValueIsZero(Op0, Negative2, 0, &I)) - BoolCast = Op0, OtherOp = Op1; - else if (MaskedValueIsZero(Op1, Negative2, 0, &I)) - BoolCast = Op1, OtherOp = Op0; + if (MaskedValueIsZero(Op0, Negative2, 0, &I)) { + BoolCast = Op0; + OtherOp = Op1; + } else if (MaskedValueIsZero(Op1, Negative2, 0, &I)) { + BoolCast = Op1; + OtherOp = Op0; + } if (BoolCast) { Value *V = Builder->CreateSub(Constant::getNullValue(I.getType()), |