From: Quentin Colombet Date: Tue, 15 Sep 2015 18:19:39 +0000 (+0000) Subject: [ShrinkWrapping] Fix an infinite loop while looking for restore point. X-Git-Url: http://plrg.eecs.uci.edu/git/?p=oota-llvm.git;a=commitdiff_plain;h=07400eaee02cc4cf136c2513e9338f6db5712f96 [ShrinkWrapping] Fix an infinite loop while looking for restore point. This may happen when the input program itself contains an infinite loop with no exit block. In that case, we would fail to find a block post-dominating the loop such that this block is outside of the loop. This fixes PR24823. Working on reducing the test case. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@247710 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/CodeGen/ShrinkWrap.cpp b/lib/CodeGen/ShrinkWrap.cpp index 7e2f15163cb..c406c149c00 100644 --- a/lib/CodeGen/ShrinkWrap.cpp +++ b/lib/CodeGen/ShrinkWrap.cpp @@ -309,6 +309,14 @@ void ShrinkWrap::updateSaveRestorePoints(MachineBasicBlock &MBB) { } } else { + // If the loop does not exit, there is no point in looking + // for a post-dominator outside the loop. + SmallVector ExitBlocks; + MLI->getLoopFor(Restore)->getExitingBlocks(ExitBlocks); + if (ExitBlocks.empty()) { + Restore = nullptr; + break; + } // Push Restore outside of this loop if immediate post-dominator is // different from restore block. If immediate post-dominator is not // different, bail out.