diff options
| author | Reid Spencer <rspencer@reidspencer.com> | 2007-03-02 00:28:52 +0000 |
|---|---|---|
| committer | Reid Spencer <rspencer@reidspencer.com> | 2007-03-02 00:28:52 +0000 |
| commit | cae5754619433aed7be74abbf1c0551a82d369cb (patch) | |
| tree | 77599913d2e900d4774e81cce3333f31f25c9fa6 /lib/Transforms/Scalar/Reassociate.cpp | |
| parent | 34da0ac28f7bfe737f92d3a098697a6ebc09a25e (diff) | |
Prefer non-virtual calls to ConstantInt::isZero over virtual calls to
Constant::isNullValue() in situations where it is possible.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@34821 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms/Scalar/Reassociate.cpp')
| -rw-r--r-- | lib/Transforms/Scalar/Reassociate.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/Transforms/Scalar/Reassociate.cpp b/lib/Transforms/Scalar/Reassociate.cpp index aa851b290f..8030a76ece 100644 --- a/lib/Transforms/Scalar/Reassociate.cpp +++ b/lib/Transforms/Scalar/Reassociate.cpp @@ -536,7 +536,7 @@ Value *Reassociate::OptimizeExpression(BinaryOperator *I, switch (Opcode) { default: break; case Instruction::And: - if (CstVal->isNullValue()) { // ... & 0 -> 0 + if (CstVal->isZero()) { // ... & 0 -> 0 ++NumAnnihil; return CstVal; } else if (CstVal->isAllOnesValue()) { // ... & -1 -> ... @@ -544,10 +544,10 @@ Value *Reassociate::OptimizeExpression(BinaryOperator *I, } break; case Instruction::Mul: - if (CstVal->isNullValue()) { // ... * 0 -> 0 + if (CstVal->isZero()) { // ... * 0 -> 0 ++NumAnnihil; return CstVal; - } else if (cast<ConstantInt>(CstVal)->isUnitValue()) { + } else if (cast<ConstantInt>(CstVal)->isOne()) { Ops.pop_back(); // ... * 1 -> ... } break; @@ -559,7 +559,7 @@ Value *Reassociate::OptimizeExpression(BinaryOperator *I, // FALLTHROUGH! case Instruction::Add: case Instruction::Xor: - if (CstVal->isNullValue()) // ... [|^+] 0 -> ... + if (CstVal->isZero()) // ... [|^+] 0 -> ... Ops.pop_back(); break; } |