summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChandler Carruth <chandlerc@gmail.com>2014-03-18 21:58:38 +0000
committerChandler Carruth <chandlerc@gmail.com>2014-03-18 21:58:38 +0000
commit633eb08d3163944e4e9d67a9c9faf05c9a3ef7f1 (patch)
tree59c5fad7dcc535dd1117e6303183383b17b8c8f6
parenta22773cd45e82e07691284abf85f0fa967eef520 (diff)
downloadllvm-633eb08d3163944e4e9d67a9c9faf05c9a3ef7f1.tar.gz
llvm-633eb08d3163944e4e9d67a9c9faf05c9a3ef7f1.tar.bz2
llvm-633eb08d3163944e4e9d67a9c9faf05c9a3ef7f1.tar.xz
[LV] The actual change I intended to commit in r204148. Sorry for the
noise. Original commit log: 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@204187 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r--lib/Transforms/Vectorize/LoopVectorize.cpp9
1 files changed, 1 insertions, 8 deletions
diff --git a/lib/Transforms/Vectorize/LoopVectorize.cpp b/lib/Transforms/Vectorize/LoopVectorize.cpp
index dd8d5fce8d..1f02bf6cbe 100644
--- a/lib/Transforms/Vectorize/LoopVectorize.cpp
+++ b/lib/Transforms/Vectorize/LoopVectorize.cpp
@@ -1064,14 +1064,7 @@ struct LoopVectorize : public FunctionPass {
}
bool processLoop(Loop *L) {
- // We only handle inner loops, so if there are children just recurse.
- if (!L->empty()) {
- bool Changed = false;
- for (Loop *InnerL : *L)
- Changed |= processLoop(InnerL);
- return Changed;
- }
-
+ assert(L->empty() && "Only process inner loops.");
DEBUG(dbgs() << "LV: Checking a loop in \"" <<
L->getHeader()->getParent()->getName() << "\"\n");