diff options
author | Dan Gohman <gohman@apple.com> | 2010-08-31 22:53:17 +0000 |
---|---|---|
committer | Dan Gohman <gohman@apple.com> | 2010-08-31 22:53:17 +0000 |
commit | 47308d5da3a6ad3237908c8d75a31472223b84a4 (patch) | |
tree | 457fa5717b0cbcb4e9731e458f22add122f95711 /llvm/lib/Analysis/ScalarEvolution.cpp | |
parent | f01a5eed1e50fcd916ccfe22d6e84f6c6ef4bb5b (diff) | |
download | bcm5719-llvm-47308d5da3a6ad3237908c8d75a31472223b84a4.tar.gz bcm5719-llvm-47308d5da3a6ad3237908c8d75a31472223b84a4.zip |
Reapply r112432, now that the real problem is addressed.
llvm-svn: 112667
Diffstat (limited to 'llvm/lib/Analysis/ScalarEvolution.cpp')
-rw-r--r-- | llvm/lib/Analysis/ScalarEvolution.cpp | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/llvm/lib/Analysis/ScalarEvolution.cpp b/llvm/lib/Analysis/ScalarEvolution.cpp index 795232b7cec..b892d85f9f4 100644 --- a/llvm/lib/Analysis/ScalarEvolution.cpp +++ b/llvm/lib/Analysis/ScalarEvolution.cpp @@ -3332,11 +3332,16 @@ const SCEV *ScalarEvolution::createSCEV(Value *V) { // LLVM IR canonical form means we need only traverse the left operands. SmallVector<const SCEV *, 4> AddOps; AddOps.push_back(getSCEV(U->getOperand(1))); - for (Value *Op = U->getOperand(0); - Op->getValueID() == Instruction::Add + Value::InstructionVal; - Op = U->getOperand(0)) { + for (Value *Op = U->getOperand(0); ; Op = U->getOperand(0)) { + unsigned Opcode = Op->getValueID() - Value::InstructionVal; + if (Opcode != Instruction::Add && Opcode != Instruction::Sub) + break; U = cast<Operator>(Op); - AddOps.push_back(getSCEV(U->getOperand(1))); + const SCEV *Op1 = getSCEV(U->getOperand(1)); + if (Opcode == Instruction::Sub) + AddOps.push_back(getNegativeSCEV(Op1)); + else + AddOps.push_back(Op1); } AddOps.push_back(getSCEV(U->getOperand(0))); return getAddExpr(AddOps); |