diff options
author | Tobias Grosser <tobias@grosser.es> | 2016-02-01 10:07:43 +0000 |
---|---|---|
committer | Tobias Grosser <tobias@grosser.es> | 2016-02-01 10:07:43 +0000 |
commit | ccbe383594fc3048bc4f6a7bc2e77c949862927f (patch) | |
tree | 06778caf06417433aa993e4cb5587e31450e666b | |
parent | 56b039ea1725d388620172c0512e015cb59383da (diff) | |
download | bcm5719-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.cpp | 9 |
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) { |