summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTobias Grosser <tobias@grosser.es>2016-02-01 10:07:43 +0000
committerTobias Grosser <tobias@grosser.es>2016-02-01 10:07:43 +0000
commitccbe383594fc3048bc4f6a7bc2e77c949862927f (patch)
tree06778caf06417433aa993e4cb5587e31450e666b
parent56b039ea1725d388620172c0512e015cb59383da (diff)
downloadbcm5719-llvm-ccbe383594fc3048bc4f6a7bc2e77c949862927f.tar.gz
bcm5719-llvm-ccbe383594fc3048bc4f6a7bc2e77c949862927f.zip
ScopInfo: Drop dead code in schedule description
In https://llvm.org/svn/llvm-project/polly/trunk@251870 code was committed to avoid a failure in the presence of infinite loops, but the test case committed along with this change passes without the actual change. I looked back into the code and also checked with the original committer (Johannes), but could not find the reason why the code is needed. The introduction of LoopStacks for buildSchedule in one of the next commits will make it even more clear that this code is not needed, but I remove this ahead of time to facilitate bisecting in case I missed something. llvm-svn: 259347
-rw-r--r--polly/lib/Analysis/ScopInfo.cpp9
1 files changed, 0 insertions, 9 deletions
diff --git a/polly/lib/Analysis/ScopInfo.cpp b/polly/lib/Analysis/ScopInfo.cpp
index 9ee586ae699..d4ad62444ea 100644
--- a/polly/lib/Analysis/ScopInfo.cpp
+++ b/polly/lib/Analysis/ScopInfo.cpp
@@ -3484,15 +3484,6 @@ void Scop::buildSchedule(
while (L && NumVisited == L->getNumBlocks()) {
auto *PL = L->getParentLoop();
- // Either we have a proper loop and we also build a schedule for the
- // parent loop or we have a infinite loop that does not have a proper
- // parent loop. In the former case this conditional will be skipped, in
- // the latter case however we will break here as we do not build a domain
- // nor a schedule for a infinite loop.
- assert(LoopSchedules.count(PL) || LSchedule == nullptr);
- if (!LoopSchedules.count(PL))
- break;
-
auto &PSchedulePair = LoopSchedules[PL];
if (LSchedule) {
OpenPOWER on IntegriCloud