diff options
author | Chris Lattner <sabre@nondot.org> | 2010-01-05 04:55:35 +0000 |
---|---|---|
committer | Chris Lattner <sabre@nondot.org> | 2010-01-05 04:55:35 +0000 |
commit | f741d72b8492daf5569508d1feabc25400d71afc (patch) | |
tree | 56d196b09ca40e0eb3554bc04c78dc3f9455b710 /llvm/lib | |
parent | d4c4fc4440a29d22d35ac37db449a0e7138f3d43 (diff) | |
download | bcm5719-llvm-f741d72b8492daf5569508d1feabc25400d71afc.tar.gz bcm5719-llvm-f741d72b8492daf5569508d1feabc25400d71afc.zip |
fix an infinite loop in reassociate building emacs.
llvm-svn: 92679
Diffstat (limited to 'llvm/lib')
-rw-r--r-- | llvm/lib/Transforms/Scalar/Reassociate.cpp | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/llvm/lib/Transforms/Scalar/Reassociate.cpp b/llvm/lib/Transforms/Scalar/Reassociate.cpp index 564036bba3e..f2a8633f97a 100644 --- a/llvm/lib/Transforms/Scalar/Reassociate.cpp +++ b/llvm/lib/Transforms/Scalar/Reassociate.cpp @@ -932,6 +932,10 @@ void Reassociate::ReassociateBB(BasicBlock *BB) { if (BI->getOpcode() == Instruction::Sub) { if (ShouldBreakUpSubtract(BI)) { BI = BreakUpSubtract(BI, ValueRankMap); + // Reset the BBI iterator in case BreakUpSubtract changed the + // instruction it points to. + BBI = BI; + ++BBI; MadeChange = true; } else if (BinaryOperator::isNeg(BI)) { // Otherwise, this is a negation. See if the operand is a multiply tree |