diff options
| author | David Greene <greened@obbligato.org> | 2007-06-29 02:53:16 +0000 |
|---|---|---|
| committer | David Greene <greened@obbligato.org> | 2007-06-29 02:53:16 +0000 |
| commit | c08fa28897356be54fba724056c3aa91da8b3e39 (patch) | |
| tree | 001400f1d5afb8809d91de5fde5b5755a261e085 /lib/Transforms | |
| parent | cfacc8f5d25f38112eee836530004f7816439765 (diff) | |
| download | external_llvm-c08fa28897356be54fba724056c3aa91da8b3e39.zip external_llvm-c08fa28897356be54fba724056c3aa91da8b3e39.tar.gz external_llvm-c08fa28897356be54fba724056c3aa91da8b3e39.tar.bz2 | |
Fix reference to iterator invalidated by an erase operation. Uncovered
by _GLIBCXX_DEBUG.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@37796 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms')
| -rw-r--r-- | lib/Transforms/Utils/LoopSimplify.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/lib/Transforms/Utils/LoopSimplify.cpp b/lib/Transforms/Utils/LoopSimplify.cpp index 8a3e525..0a5de2b 100644 --- a/lib/Transforms/Utils/LoopSimplify.cpp +++ b/lib/Transforms/Utils/LoopSimplify.cpp @@ -564,11 +564,12 @@ Loop *LoopSimplify::SeparateNestedLoop(Loop *L) { // Scan all of the loop children of L, moving them to OuterLoop if they are // not part of the inner loop. - for (Loop::iterator I = L->begin(); I != L->end(); ) - if (BlocksInL.count((*I)->getHeader())) + const std::vector<Loop*> &SubLoops = L->getSubLoops(); + for (size_t I = 0; I != SubLoops.size(); ) + if (BlocksInL.count(SubLoops[I]->getHeader())) ++I; // Loop remains in L else - NewOuter->addChildLoop(L->removeChildLoop(I)); + NewOuter->addChildLoop(L->removeChildLoop(SubLoops.begin() + I)); // Now that we know which blocks are in L and which need to be moved to // OuterLoop, move any blocks that need it. |