From 697483addf056595c997302f1316cc59244eefaa Mon Sep 17 00:00:00 2001 From: Jakob Stoklund Olesen Date: Wed, 15 Dec 2010 17:49:52 +0000 Subject: [PATCH] Detect and enumerate bypass loops. Bypass loops have the current live range live through, but contain no uses or defs. Splitting around a bypass loop can free registers for other uses inside the loop by spilling the split range. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@121871 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/CodeGen/SplitKit.cpp | 30 ++++++++++++++++++++++++++++++ lib/CodeGen/SplitKit.h | 9 +++++++++ 2 files changed, 39 insertions(+) diff --git a/lib/CodeGen/SplitKit.cpp b/lib/CodeGen/SplitKit.cpp index fd18729b85b..e14251ee3d1 100644 --- a/lib/CodeGen/SplitKit.cpp +++ b/lib/CodeGen/SplitKit.cpp @@ -325,6 +325,36 @@ const MachineLoop *SplitAnalysis::getBestSplitLoop() { return Best; } +/// isBypassLoop - Return true if curli is live through Loop and has no uses +/// inside the loop. Bypass loops are candidates for splitting because it can +/// prevent interference inside the loop. +bool SplitAnalysis::isBypassLoop(const MachineLoop *Loop) { + // If curli is live into the loop header and there are no uses in the loop, it + // must be live in the entire loop and live on at least one exiting edge. + return !usingLoops_.count(Loop) && + lis_.isLiveInToMBB(*curli_, Loop->getHeader()); +} + +/// getBypassLoops - Get all the maximal bypass loops. These are the bypass +/// loops whose parent is not a bypass loop. +void SplitAnalysis::getBypassLoops(LoopPtrSet &BypassLoops) { + SmallVector Todo(loops_.begin(), loops_.end()); + while (!Todo.empty) { + MachineLoop *Loop = Todo.pop_back_val(); + if (!usingLoops_.count(Loop)) { + // This is either a bypass loop or completely irrelevant. + if (lis_.isLiveInToMBB(*curli_, Loop->getHeader())) + BypassLoops.insert(Loop); + // Either way, skip the child loops. + continue; + } + + // The child loops may be bypass loops. + Todo.append(Loop->begin(), Loop->end()); + } +} + + //===----------------------------------------------------------------------===// // LiveIntervalMap //===----------------------------------------------------------------------===// diff --git a/lib/CodeGen/SplitKit.h b/lib/CodeGen/SplitKit.h index f290eb64610..0930b3974cd 100644 --- a/lib/CodeGen/SplitKit.h +++ b/lib/CodeGen/SplitKit.h @@ -141,6 +141,15 @@ public: /// separate register, or NULL. const MachineLoop *getBestSplitLoop(); + /// isBypassLoop - Return true if curli is live through Loop and has no uses + /// inside the loop. Bypass loops are candidates for splitting because it can + /// prevent interference inside the loop. + bool isBypassLoop(const MachineLoop *Loop); + + /// getBypassLoops - Get all the maximal bypass loops. These are the bypass + /// loops whose parent is not a bypass loop. + void getBypassLoops(LoopPtrSet&); + /// getMultiUseBlocks - Add basic blocks to Blocks that may benefit from /// having curli split to a new live interval. Return true if Blocks can be /// passed to SplitEditor::splitSingleBlocks.