diff options
| author | Chris Lattner <sabre@nondot.org> | 2010-01-09 06:01:36 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2010-01-09 06:01:36 +0000 |
| commit | ab7087ad66e6bb994dd17c644bad7641f492c9cf (patch) | |
| tree | 446d18c0e411c697f1bf7310ad0fd05e21064d42 /llvm/lib/Transforms/Scalar/Reassociate.cpp | |
| parent | 002ba6b4d0d944d26d18e452104abfb2810a22c2 (diff) | |
| download | llvm-ab7087ad66e6bb994dd17c644bad7641f492c9cf.zip llvm-ab7087ad66e6bb994dd17c644bad7641f492c9cf.tar.gz llvm-ab7087ad66e6bb994dd17c644bad7641f492c9cf.tar.bz2 | |
only factor from expressions whose uses are empty and whose
base is the right expression type. This fixes PR5981.
llvm-svn: 93045
Diffstat (limited to 'llvm/lib/Transforms/Scalar/Reassociate.cpp')
| -rw-r--r-- | llvm/lib/Transforms/Scalar/Reassociate.cpp | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/llvm/lib/Transforms/Scalar/Reassociate.cpp b/llvm/lib/Transforms/Scalar/Reassociate.cpp index 3cc380a..4a99f4a 100644 --- a/llvm/lib/Transforms/Scalar/Reassociate.cpp +++ b/llvm/lib/Transforms/Scalar/Reassociate.cpp @@ -792,6 +792,11 @@ Value *Reassociate::OptimizeAdd(Instruction *I, Instruction *DummyInst = BinaryOperator::CreateAdd(MaxOccVal, MaxOccVal); SmallVector<Value*, 4> NewMulOps; for (unsigned i = 0, e = Ops.size(); i != e; ++i) { + // Only try to remove factors from expressions we're allowed to. + BinaryOperator *BOp = dyn_cast<BinaryOperator>(Ops[i].Op); + if (BOp == 0 || BOp->getOpcode() != Instruction::Mul || !BOp->use_empty()) + continue; + if (Value *V = RemoveFactorFromExpression(Ops[i].Op, MaxOccVal)) { NewMulOps.push_back(V); Ops.erase(Ops.begin()+i); |