diff options
author | David Majnemer <david.majnemer@gmail.com> | 2015-04-18 04:41:30 +0000 |
---|---|---|
committer | David Majnemer <david.majnemer@gmail.com> | 2015-04-18 04:41:30 +0000 |
commit | 45951a662644f5a5e021806e8a75984115b9ebef (patch) | |
tree | 54d70f816966da0010e265f243109b36c884f099 /llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp | |
parent | 279e3ee954fd41164fd78e4f20ea54a3c53cc5c8 (diff) | |
download | llvm-45951a662644f5a5e021806e8a75984115b9ebef.zip llvm-45951a662644f5a5e021806e8a75984115b9ebef.tar.gz llvm-45951a662644f5a5e021806e8a75984115b9ebef.tar.bz2 |
[InstCombine] (mul nsw 1, INT_MIN) != (shl nsw 1, 31)
Multiplying INT_MIN by 1 doesn't trigger nsw. However, shifting 1 into
the sign bit *does* trigger nsw.
llvm-svn: 235250
Diffstat (limited to 'llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp')
-rw-r--r-- | llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp b/llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp index 35513f1..a554e9f 100644 --- a/llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp +++ b/llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp @@ -217,12 +217,16 @@ Instruction *InstCombiner::visitMul(BinaryOperator &I) { NewCst = getLogBase2Vector(CV); if (NewCst) { + unsigned Width = NewCst->getType()->getPrimitiveSizeInBits(); BinaryOperator *Shl = BinaryOperator::CreateShl(NewOp, NewCst); if (I.hasNoUnsignedWrap()) Shl->setHasNoUnsignedWrap(); - if (I.hasNoSignedWrap() && NewCst->isNotMinSignedValue()) - Shl->setHasNoSignedWrap(); + if (I.hasNoSignedWrap()) { + uint64_t V; + if (match(NewCst, m_ConstantInt(V)) && V != Width - 1) + Shl->setHasNoSignedWrap(); + } return Shl; } |