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 | 5f94af0f725690a75b18909da2ff8cee0f93bebb (patch) | |
| tree | cc25ece030ec40877d867322ab7ddec183154ad8 /lib/Transforms/Scalar/Reassociate.cpp | |
| parent | 2abc1ec2dd57511638b478c34729b9ca6f1d6ebc (diff) | |
fix an infinite loop in reassociate building emacs.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@92679 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms/Scalar/Reassociate.cpp')
| -rw-r--r-- | lib/Transforms/Scalar/Reassociate.cpp | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/lib/Transforms/Scalar/Reassociate.cpp b/lib/Transforms/Scalar/Reassociate.cpp index 564036bba3..f2a8633f97 100644 --- a/lib/Transforms/Scalar/Reassociate.cpp +++ b/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 |