From ecfe9d06ebf5474fb40de27ed883d2b4277e38f4 Mon Sep 17 00:00:00 2001 From: Tobias Grosser Date: Thu, 8 May 2014 07:12:44 +0000 Subject: [PATCH] SCEV: Use I = vector<>.erase(I) to iterate and delete at the same time llvm-svn: 208282 --- llvm/lib/Analysis/ScalarEvolution.cpp | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/llvm/lib/Analysis/ScalarEvolution.cpp b/llvm/lib/Analysis/ScalarEvolution.cpp index 148e58d548f1..8f5a2a98cc16 100644 --- a/llvm/lib/Analysis/ScalarEvolution.cpp +++ b/llvm/lib/Analysis/ScalarEvolution.cpp @@ -7223,12 +7223,9 @@ static void findArrayDimensionsRec(ScalarEvolution &SE, Terms[I] = Q; } - // Remove all SCEVConstants. - for (unsigned I = 0; I < Terms.size();) - if (isa(Terms[I])) - Terms.erase(Terms.begin() + I); - else - ++I; + for (auto I = Terms.begin(), E = Terms.end(); I != E; I++) + if (isa(*I)) + I = Terms.erase(I); if (Terms.size() > 0) findArrayDimensionsRec(SE, Terms, Sizes, Zero, One);