From: Chandler Carruth Date: Tue, 18 Mar 2014 21:51:46 +0000 (+0000) Subject: [LV] Replace some dead code with an assert. When I first ported this X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=a22773cd45e82e07691284abf85f0fa967eef520;p=oota-llvm.git [LV] Replace some dead code with an assert. When I first ported this pass from a loop pass to a function pass I did so in the naive, recursive way. It doesn't actually work, we need a worklist instead. When I switched to the worklist I didn't delete the naive recursion. That recursion was also buggy because it was dead and never really exercised. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@204184 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Transforms/Vectorize/LoopVectorize.cpp b/lib/Transforms/Vectorize/LoopVectorize.cpp index 435c0054d0d..dd8d5fce8d4 100644 --- a/lib/Transforms/Vectorize/LoopVectorize.cpp +++ b/lib/Transforms/Vectorize/LoopVectorize.cpp @@ -1067,8 +1067,8 @@ struct LoopVectorize : public FunctionPass { // We only handle inner loops, so if there are children just recurse. if (!L->empty()) { bool Changed = false; - for (Loop::iterator I = L->begin(), E = L->begin(); I != E; ++I) - Changed |= processLoop(*I); + for (Loop *InnerL : *L) + Changed |= processLoop(InnerL); return Changed; }