blob: 3c139068e6836b84234f7efcf188c63b90bb1b3d [file] [log] [blame]
Owen Anderson0bda0e82007-10-31 03:37:57 +00001//===- StrongPhiElimination.cpp - Eliminate PHI nodes by inserting copies -===//
2//
3// The LLVM Compiler Infrastructure
4//
Chris Lattner4ee451d2007-12-29 20:36:04 +00005// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
Owen Anderson0bda0e82007-10-31 03:37:57 +00007//
8//===----------------------------------------------------------------------===//
9//
10// This pass eliminates machine instruction PHI nodes by inserting copy
11// instructions, using an intelligent copy-folding technique based on
12// dominator information. This is technique is derived from:
13//
14// Budimlic, et al. Fast copy coalescing and live-range identification.
15// In Proceedings of the ACM SIGPLAN 2002 Conference on Programming Language
16// Design and Implementation (Berlin, Germany, June 17 - 19, 2002).
17// PLDI '02. ACM, New York, NY, 25-32.
18// DOI= http://doi.acm.org/10.1145/512529.512534
19//
20//===----------------------------------------------------------------------===//
21
22#define DEBUG_TYPE "strongphielim"
23#include "llvm/CodeGen/Passes.h"
Owen Andersoneb37ecc2008-03-10 07:22:36 +000024#include "llvm/CodeGen/LiveIntervalAnalysis.h"
Owen Anderson0bda0e82007-10-31 03:37:57 +000025#include "llvm/CodeGen/MachineDominators.h"
26#include "llvm/CodeGen/MachineFunctionPass.h"
27#include "llvm/CodeGen/MachineInstr.h"
Owen Anderson755ebab2008-03-17 06:08:26 +000028#include "llvm/CodeGen/MachineLoopInfo.h"
Owen Anderson0d893b42008-01-08 05:16:15 +000029#include "llvm/CodeGen/MachineRegisterInfo.h"
Owen Anderson3947e4d2008-05-30 18:38:26 +000030#include "llvm/CodeGen/RegisterCoalescer.h"
Owen Anderson0bda0e82007-10-31 03:37:57 +000031#include "llvm/Target/TargetInstrInfo.h"
32#include "llvm/Target/TargetMachine.h"
Owen Andersonefbcebc2007-12-23 15:37:26 +000033#include "llvm/ADT/DepthFirstIterator.h"
Owen Anderson0bda0e82007-10-31 03:37:57 +000034#include "llvm/ADT/Statistic.h"
Owen Andersone04cec02008-09-22 21:58:58 +000035#include "llvm/Support/Debug.h"
Owen Anderson0bda0e82007-10-31 03:37:57 +000036using namespace llvm;
37
Owen Anderson0bda0e82007-10-31 03:37:57 +000038namespace {
Nick Lewycky6726b6d2009-10-25 06:33:48 +000039 struct StrongPHIElimination : public MachineFunctionPass {
Owen Anderson0bda0e82007-10-31 03:37:57 +000040 static char ID; // Pass identification, replacement for typeid
Dan Gohmanae73dc12008-09-04 17:05:41 +000041 StrongPHIElimination() : MachineFunctionPass(&ID) {}
Owen Anderson0bda0e82007-10-31 03:37:57 +000042
Owen Andersonec1213f2008-01-09 22:40:54 +000043 // Waiting stores, for each MBB, the set of copies that need to
44 // be inserted into that MBB
Owen Andersonafc6de02007-12-10 08:07:09 +000045 DenseMap<MachineBasicBlock*,
Owen Anderson8481ee72008-10-02 19:40:33 +000046 std::multimap<unsigned, unsigned> > Waiting;
Owen Andersonefbcebc2007-12-23 15:37:26 +000047
Owen Andersonec1213f2008-01-09 22:40:54 +000048 // Stacks holds the renaming stack for each register
Owen Andersonefbcebc2007-12-23 15:37:26 +000049 std::map<unsigned, std::vector<unsigned> > Stacks;
Owen Andersonec1213f2008-01-09 22:40:54 +000050
51 // Registers in UsedByAnother are PHI nodes that are themselves
52 // used as operands to another another PHI node
Owen Andersonefbcebc2007-12-23 15:37:26 +000053 std::set<unsigned> UsedByAnother;
Owen Andersonec1213f2008-01-09 22:40:54 +000054
Owen Andersona9efb262008-06-05 17:22:53 +000055 // RenameSets are the is a map from a PHI-defined register
Owen Andersonc12417e2008-07-24 17:12:16 +000056 // to the input registers to be coalesced along with the
57 // predecessor block for those input registers.
58 std::map<unsigned, std::map<unsigned, MachineBasicBlock*> > RenameSets;
Owen Andersondfd07ea2008-03-12 04:22:57 +000059
60 // PhiValueNumber holds the ID numbers of the VNs for each phi that we're
61 // eliminating, indexed by the register defined by that phi.
62 std::map<unsigned, unsigned> PhiValueNumber;
Owen Andersonafc6de02007-12-10 08:07:09 +000063
Owen Andersonec1213f2008-01-09 22:40:54 +000064 // Store the DFS-in number of each block
65 DenseMap<MachineBasicBlock*, unsigned> preorder;
66
67 // Store the DFS-out number of each block
68 DenseMap<MachineBasicBlock*, unsigned> maxpreorder;
69
Owen Andersona4ad2e72007-11-06 04:49:43 +000070 bool runOnMachineFunction(MachineFunction &Fn);
71
Owen Anderson0bda0e82007-10-31 03:37:57 +000072 virtual void getAnalysisUsage(AnalysisUsage &AU) const {
Dan Gohman845012e2009-07-31 23:37:33 +000073 AU.setPreservesCFG();
Owen Anderson0bda0e82007-10-31 03:37:57 +000074 AU.addRequired<MachineDominatorTree>();
Lang Hames233a60e2009-11-03 23:52:08 +000075 AU.addRequired<SlotIndexes>();
76 AU.addPreserved<SlotIndexes>();
Owen Andersoneb37ecc2008-03-10 07:22:36 +000077 AU.addRequired<LiveIntervals>();
78
79 // TODO: Actually make this true.
80 AU.addPreserved<LiveIntervals>();
Owen Anderson3947e4d2008-05-30 18:38:26 +000081 AU.addPreserved<RegisterCoalescer>();
Owen Anderson0bda0e82007-10-31 03:37:57 +000082 MachineFunctionPass::getAnalysisUsage(AU);
83 }
84
85 virtual void releaseMemory() {
86 preorder.clear();
87 maxpreorder.clear();
Owen Andersona4ad2e72007-11-06 04:49:43 +000088
Owen Andersonefbcebc2007-12-23 15:37:26 +000089 Waiting.clear();
Owen Andersonec1213f2008-01-09 22:40:54 +000090 Stacks.clear();
91 UsedByAnother.clear();
92 RenameSets.clear();
Owen Anderson0bda0e82007-10-31 03:37:57 +000093 }
94
95 private:
Owen Andersonec1213f2008-01-09 22:40:54 +000096
97 /// DomForestNode - Represents a node in the "dominator forest". This is
98 /// a forest in which the nodes represent registers and the edges
99 /// represent a dominance relation in the block defining those registers.
Owen Anderson83430bc2007-11-04 22:33:26 +0000100 struct DomForestNode {
101 private:
Owen Andersonec1213f2008-01-09 22:40:54 +0000102 // Store references to our children
Owen Anderson83430bc2007-11-04 22:33:26 +0000103 std::vector<DomForestNode*> children;
Owen Andersonec1213f2008-01-09 22:40:54 +0000104 // The register we represent
Owen Andersonee49b532007-11-06 05:22:43 +0000105 unsigned reg;
Owen Anderson83430bc2007-11-04 22:33:26 +0000106
Owen Andersonec1213f2008-01-09 22:40:54 +0000107 // Add another node as our child
Owen Anderson83430bc2007-11-04 22:33:26 +0000108 void addChild(DomForestNode* DFN) { children.push_back(DFN); }
109
110 public:
111 typedef std::vector<DomForestNode*>::iterator iterator;
112
Owen Andersonec1213f2008-01-09 22:40:54 +0000113 // Create a DomForestNode by providing the register it represents, and
114 // the node to be its parent. The virtual root node has register 0
115 // and a null parent.
Owen Andersonee49b532007-11-06 05:22:43 +0000116 DomForestNode(unsigned r, DomForestNode* parent) : reg(r) {
Owen Anderson83430bc2007-11-04 22:33:26 +0000117 if (parent)
118 parent->addChild(this);
119 }
120
Owen Andersona4ad2e72007-11-06 04:49:43 +0000121 ~DomForestNode() {
122 for (iterator I = begin(), E = end(); I != E; ++I)
123 delete *I;
124 }
Owen Anderson83430bc2007-11-04 22:33:26 +0000125
Owen Andersonec1213f2008-01-09 22:40:54 +0000126 /// getReg - Return the regiser that this node represents
Owen Andersonee49b532007-11-06 05:22:43 +0000127 inline unsigned getReg() { return reg; }
Owen Andersona4ad2e72007-11-06 04:49:43 +0000128
Owen Andersonec1213f2008-01-09 22:40:54 +0000129 // Provide iterator access to our children
Owen Andersona4ad2e72007-11-06 04:49:43 +0000130 inline DomForestNode::iterator begin() { return children.begin(); }
131 inline DomForestNode::iterator end() { return children.end(); }
Owen Anderson83430bc2007-11-04 22:33:26 +0000132 };
133
Owen Anderson0bda0e82007-10-31 03:37:57 +0000134 void computeDFS(MachineFunction& MF);
Owen Anderson60a877d2007-11-07 05:17:15 +0000135 void processBlock(MachineBasicBlock* MBB);
Owen Anderson83430bc2007-11-04 22:33:26 +0000136
Owen Andersonc12417e2008-07-24 17:12:16 +0000137 std::vector<DomForestNode*> computeDomForest(
138 std::map<unsigned, MachineBasicBlock*>& instrs,
Owen Andersonddd060f2008-01-10 01:36:43 +0000139 MachineRegisterInfo& MRI);
Owen Andersond525f662007-12-11 20:12:11 +0000140 void processPHIUnion(MachineInstr* Inst,
Owen Andersonc12417e2008-07-24 17:12:16 +0000141 std::map<unsigned, MachineBasicBlock*>& PHIUnion,
Owen Anderson62d67dd2007-12-13 05:53:03 +0000142 std::vector<StrongPHIElimination::DomForestNode*>& DF,
143 std::vector<std::pair<unsigned, unsigned> >& locals);
Owen Andersonf1519e82007-12-24 22:12:23 +0000144 void ScheduleCopies(MachineBasicBlock* MBB, std::set<unsigned>& pushed);
Owen Anderson724acf12008-08-07 18:28:07 +0000145 void InsertCopies(MachineDomTreeNode* MBB,
Owen Andersone46611e2008-03-24 04:11:27 +0000146 SmallPtrSet<MachineBasicBlock*, 16>& v);
Owen Andersonfae86ed2008-09-23 04:37:10 +0000147 bool mergeLiveIntervals(unsigned primary, unsigned secondary);
Owen Anderson0bda0e82007-10-31 03:37:57 +0000148 };
Owen Anderson0bda0e82007-10-31 03:37:57 +0000149}
150
Dan Gohman844731a2008-05-13 00:00:25 +0000151char StrongPHIElimination::ID = 0;
152static RegisterPass<StrongPHIElimination>
153X("strong-phi-node-elimination",
154 "Eliminate PHI nodes for register allocation, intelligently");
155
Dan Gohman6ddba2b2008-05-13 02:05:11 +0000156const PassInfo *const llvm::StrongPHIEliminationID = &X;
Owen Anderson0bda0e82007-10-31 03:37:57 +0000157
158/// computeDFS - Computes the DFS-in and DFS-out numbers of the dominator tree
159/// of the given MachineFunction. These numbers are then used in other parts
160/// of the PHI elimination process.
161void StrongPHIElimination::computeDFS(MachineFunction& MF) {
162 SmallPtrSet<MachineDomTreeNode*, 8> frontier;
163 SmallPtrSet<MachineDomTreeNode*, 8> visited;
164
165 unsigned time = 0;
166
167 MachineDominatorTree& DT = getAnalysis<MachineDominatorTree>();
168
169 MachineDomTreeNode* node = DT.getRootNode();
170
171 std::vector<MachineDomTreeNode*> worklist;
172 worklist.push_back(node);
173
174 while (!worklist.empty()) {
175 MachineDomTreeNode* currNode = worklist.back();
176
177 if (!frontier.count(currNode)) {
178 frontier.insert(currNode);
179 ++time;
180 preorder.insert(std::make_pair(currNode->getBlock(), time));
181 }
182
183 bool inserted = false;
Owen Andersonbc91bd32008-03-31 01:39:20 +0000184 for (MachineDomTreeNode::iterator I = currNode->begin(), E = currNode->end();
Owen Anderson0bda0e82007-10-31 03:37:57 +0000185 I != E; ++I)
186 if (!frontier.count(*I) && !visited.count(*I)) {
187 worklist.push_back(*I);
188 inserted = true;
189 break;
190 }
191
192 if (!inserted) {
193 frontier.erase(currNode);
194 visited.insert(currNode);
195 maxpreorder.insert(std::make_pair(currNode->getBlock(), time));
196
197 worklist.pop_back();
198 }
199 }
Duncan Sands1bd32712007-10-31 08:49:24 +0000200}
Owen Anderson83430bc2007-11-04 22:33:26 +0000201
Dan Gohman844731a2008-05-13 00:00:25 +0000202namespace {
203
Owen Anderson8b96b9f2007-11-06 05:26:02 +0000204/// PreorderSorter - a helper class that is used to sort registers
205/// according to the preorder number of their defining blocks
Owen Anderson83430bc2007-11-04 22:33:26 +0000206class PreorderSorter {
207private:
208 DenseMap<MachineBasicBlock*, unsigned>& preorder;
Owen Andersonddd060f2008-01-10 01:36:43 +0000209 MachineRegisterInfo& MRI;
Owen Anderson83430bc2007-11-04 22:33:26 +0000210
211public:
Owen Andersonee49b532007-11-06 05:22:43 +0000212 PreorderSorter(DenseMap<MachineBasicBlock*, unsigned>& p,
Owen Andersonddd060f2008-01-10 01:36:43 +0000213 MachineRegisterInfo& M) : preorder(p), MRI(M) { }
Owen Anderson83430bc2007-11-04 22:33:26 +0000214
Owen Andersonee49b532007-11-06 05:22:43 +0000215 bool operator()(unsigned A, unsigned B) {
Owen Anderson83430bc2007-11-04 22:33:26 +0000216 if (A == B)
217 return false;
218
Owen Andersonddd060f2008-01-10 01:36:43 +0000219 MachineBasicBlock* ABlock = MRI.getVRegDef(A)->getParent();
220 MachineBasicBlock* BBlock = MRI.getVRegDef(B)->getParent();
Owen Andersonee49b532007-11-06 05:22:43 +0000221
222 if (preorder[ABlock] < preorder[BBlock])
Owen Anderson83430bc2007-11-04 22:33:26 +0000223 return true;
Owen Andersonee49b532007-11-06 05:22:43 +0000224 else if (preorder[ABlock] > preorder[BBlock])
Owen Anderson83430bc2007-11-04 22:33:26 +0000225 return false;
226
Owen Andersonee49b532007-11-06 05:22:43 +0000227 return false;
Owen Anderson83430bc2007-11-04 22:33:26 +0000228 }
229};
230
Dan Gohman844731a2008-05-13 00:00:25 +0000231}
232
Owen Anderson8b96b9f2007-11-06 05:26:02 +0000233/// computeDomForest - compute the subforest of the DomTree corresponding
234/// to the defining blocks of the registers in question
Owen Anderson83430bc2007-11-04 22:33:26 +0000235std::vector<StrongPHIElimination::DomForestNode*>
Owen Andersonc12417e2008-07-24 17:12:16 +0000236StrongPHIElimination::computeDomForest(
237 std::map<unsigned, MachineBasicBlock*>& regs,
Owen Andersonddd060f2008-01-10 01:36:43 +0000238 MachineRegisterInfo& MRI) {
Owen Andersonec1213f2008-01-09 22:40:54 +0000239 // Begin by creating a virtual root node, since the actual results
240 // may well be a forest. Assume this node has maximum DFS-out number.
Owen Anderson83430bc2007-11-04 22:33:26 +0000241 DomForestNode* VirtualRoot = new DomForestNode(0, 0);
242 maxpreorder.insert(std::make_pair((MachineBasicBlock*)0, ~0UL));
243
Owen Andersonec1213f2008-01-09 22:40:54 +0000244 // Populate a worklist with the registers
Owen Andersonee49b532007-11-06 05:22:43 +0000245 std::vector<unsigned> worklist;
246 worklist.reserve(regs.size());
Owen Andersonc12417e2008-07-24 17:12:16 +0000247 for (std::map<unsigned, MachineBasicBlock*>::iterator I = regs.begin(),
248 E = regs.end(); I != E; ++I)
Owen Anderson00316712008-03-12 03:13:29 +0000249 worklist.push_back(I->first);
Owen Andersonee49b532007-11-06 05:22:43 +0000250
Owen Andersonec1213f2008-01-09 22:40:54 +0000251 // Sort the registers by the DFS-in number of their defining block
Owen Andersonddd060f2008-01-10 01:36:43 +0000252 PreorderSorter PS(preorder, MRI);
Owen Anderson83430bc2007-11-04 22:33:26 +0000253 std::sort(worklist.begin(), worklist.end(), PS);
254
Owen Andersonec1213f2008-01-09 22:40:54 +0000255 // Create a "current parent" stack, and put the virtual root on top of it
Owen Anderson83430bc2007-11-04 22:33:26 +0000256 DomForestNode* CurrentParent = VirtualRoot;
257 std::vector<DomForestNode*> stack;
258 stack.push_back(VirtualRoot);
259
Owen Andersonec1213f2008-01-09 22:40:54 +0000260 // Iterate over all the registers in the previously computed order
Owen Andersonee49b532007-11-06 05:22:43 +0000261 for (std::vector<unsigned>::iterator I = worklist.begin(), E = worklist.end();
262 I != E; ++I) {
Owen Andersonddd060f2008-01-10 01:36:43 +0000263 unsigned pre = preorder[MRI.getVRegDef(*I)->getParent()];
Owen Andersoncb7d9492008-01-09 06:19:05 +0000264 MachineBasicBlock* parentBlock = CurrentParent->getReg() ?
Owen Andersonddd060f2008-01-10 01:36:43 +0000265 MRI.getVRegDef(CurrentParent->getReg())->getParent() :
Owen Andersoncb7d9492008-01-09 06:19:05 +0000266 0;
Owen Andersonee49b532007-11-06 05:22:43 +0000267
Owen Andersonec1213f2008-01-09 22:40:54 +0000268 // If the DFS-in number of the register is greater than the DFS-out number
269 // of the current parent, repeatedly pop the parent stack until it isn't.
Owen Andersonee49b532007-11-06 05:22:43 +0000270 while (pre > maxpreorder[parentBlock]) {
Owen Anderson83430bc2007-11-04 22:33:26 +0000271 stack.pop_back();
272 CurrentParent = stack.back();
Owen Andersonee49b532007-11-06 05:22:43 +0000273
Owen Anderson864e3a32008-01-09 10:41:39 +0000274 parentBlock = CurrentParent->getReg() ?
Owen Andersonddd060f2008-01-10 01:36:43 +0000275 MRI.getVRegDef(CurrentParent->getReg())->getParent() :
Owen Anderson864e3a32008-01-09 10:41:39 +0000276 0;
Owen Anderson83430bc2007-11-04 22:33:26 +0000277 }
278
Owen Andersonec1213f2008-01-09 22:40:54 +0000279 // Now that we've found the appropriate parent, create a DomForestNode for
280 // this register and attach it to the forest
Owen Anderson83430bc2007-11-04 22:33:26 +0000281 DomForestNode* child = new DomForestNode(*I, CurrentParent);
Owen Andersonec1213f2008-01-09 22:40:54 +0000282
283 // Push this new node on the "current parent" stack
Owen Anderson83430bc2007-11-04 22:33:26 +0000284 stack.push_back(child);
285 CurrentParent = child;
286 }
287
Owen Andersonec1213f2008-01-09 22:40:54 +0000288 // Return a vector containing the children of the virtual root node
Owen Anderson83430bc2007-11-04 22:33:26 +0000289 std::vector<DomForestNode*> ret;
290 ret.insert(ret.end(), VirtualRoot->begin(), VirtualRoot->end());
291 return ret;
292}
Owen Andersona4ad2e72007-11-06 04:49:43 +0000293
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000294/// isLiveIn - helper method that determines, from a regno, if a register
Owen Anderson60a877d2007-11-07 05:17:15 +0000295/// is live into a block
Owen Andersonddd060f2008-01-10 01:36:43 +0000296static bool isLiveIn(unsigned r, MachineBasicBlock* MBB,
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000297 LiveIntervals& LI) {
298 LiveInterval& I = LI.getOrCreateInterval(r);
Lang Hames233a60e2009-11-03 23:52:08 +0000299 SlotIndex idx = LI.getMBBStartIdx(MBB);
Owen Andersond55fccd2008-10-12 20:39:30 +0000300 return I.liveAt(idx);
Owen Anderson60a877d2007-11-07 05:17:15 +0000301}
302
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000303/// isLiveOut - help method that determines, from a regno, if a register is
Owen Anderson60a877d2007-11-07 05:17:15 +0000304/// live out of a block.
Owen Andersonddd060f2008-01-10 01:36:43 +0000305static bool isLiveOut(unsigned r, MachineBasicBlock* MBB,
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000306 LiveIntervals& LI) {
307 for (MachineBasicBlock::succ_iterator PI = MBB->succ_begin(),
Owen Anderson724acf12008-08-07 18:28:07 +0000308 E = MBB->succ_end(); PI != E; ++PI)
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000309 if (isLiveIn(r, *PI, LI))
310 return true;
Owen Anderson60a877d2007-11-07 05:17:15 +0000311
312 return false;
313}
314
Owen Anderson87a702b2007-12-16 05:44:27 +0000315/// interferes - checks for local interferences by scanning a block. The only
316/// trick parameter is 'mode' which tells it the relationship of the two
317/// registers. 0 - defined in the same block, 1 - first properly dominates
318/// second, 2 - second properly dominates first
Owen Andersonb199cbe2008-01-10 00:33:11 +0000319static bool interferes(unsigned a, unsigned b, MachineBasicBlock* scan,
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000320 LiveIntervals& LV, unsigned mode) {
Owen Anderson87a702b2007-12-16 05:44:27 +0000321 MachineInstr* def = 0;
322 MachineInstr* kill = 0;
323
Owen Andersonddd060f2008-01-10 01:36:43 +0000324 // The code is still in SSA form at this point, so there is only one
325 // definition per VReg. Thus we can safely use MRI->getVRegDef().
326 const MachineRegisterInfo* MRI = &scan->getParent()->getRegInfo();
Owen Andersonb199cbe2008-01-10 00:33:11 +0000327
Owen Anderson87a702b2007-12-16 05:44:27 +0000328 bool interference = false;
329
330 // Wallk the block, checking for interferences
331 for (MachineBasicBlock::iterator MBI = scan->begin(), MBE = scan->end();
332 MBI != MBE; ++MBI) {
333 MachineInstr* curr = MBI;
334
335 // Same defining block...
336 if (mode == 0) {
Owen Andersonddd060f2008-01-10 01:36:43 +0000337 if (curr == MRI->getVRegDef(a)) {
338 // If we find our first definition, save it
Owen Anderson87a702b2007-12-16 05:44:27 +0000339 if (!def) {
340 def = curr;
Owen Andersonddd060f2008-01-10 01:36:43 +0000341 // If there's already an unkilled definition, then
Owen Anderson87a702b2007-12-16 05:44:27 +0000342 // this is an interference
343 } else if (!kill) {
344 interference = true;
345 break;
Owen Andersonddd060f2008-01-10 01:36:43 +0000346 // If there's a definition followed by a KillInst, then
Owen Anderson87a702b2007-12-16 05:44:27 +0000347 // they can't interfere
348 } else {
349 interference = false;
350 break;
351 }
352 // Symmetric with the above
Owen Andersonddd060f2008-01-10 01:36:43 +0000353 } else if (curr == MRI->getVRegDef(b)) {
Owen Anderson87a702b2007-12-16 05:44:27 +0000354 if (!def) {
355 def = curr;
356 } else if (!kill) {
357 interference = true;
358 break;
359 } else {
360 interference = false;
361 break;
362 }
Owen Andersonddd060f2008-01-10 01:36:43 +0000363 // Store KillInsts if they match up with the definition
Evan Cheng6130f662008-03-05 00:59:57 +0000364 } else if (curr->killsRegister(a)) {
Owen Andersonddd060f2008-01-10 01:36:43 +0000365 if (def == MRI->getVRegDef(a)) {
Owen Anderson87a702b2007-12-16 05:44:27 +0000366 kill = curr;
Evan Cheng6130f662008-03-05 00:59:57 +0000367 } else if (curr->killsRegister(b)) {
Owen Andersonddd060f2008-01-10 01:36:43 +0000368 if (def == MRI->getVRegDef(b)) {
Owen Anderson87a702b2007-12-16 05:44:27 +0000369 kill = curr;
370 }
371 }
372 }
373 // First properly dominates second...
374 } else if (mode == 1) {
Owen Andersonddd060f2008-01-10 01:36:43 +0000375 if (curr == MRI->getVRegDef(b)) {
376 // Definition of second without kill of first is an interference
Owen Anderson87a702b2007-12-16 05:44:27 +0000377 if (!kill) {
378 interference = true;
379 break;
Owen Andersonddd060f2008-01-10 01:36:43 +0000380 // Definition after a kill is a non-interference
Owen Anderson87a702b2007-12-16 05:44:27 +0000381 } else {
382 interference = false;
383 break;
384 }
385 // Save KillInsts of First
Evan Cheng6130f662008-03-05 00:59:57 +0000386 } else if (curr->killsRegister(a)) {
Owen Anderson87a702b2007-12-16 05:44:27 +0000387 kill = curr;
388 }
389 // Symmetric with the above
390 } else if (mode == 2) {
Owen Andersonddd060f2008-01-10 01:36:43 +0000391 if (curr == MRI->getVRegDef(a)) {
Owen Anderson87a702b2007-12-16 05:44:27 +0000392 if (!kill) {
393 interference = true;
394 break;
395 } else {
396 interference = false;
397 break;
398 }
Evan Cheng6130f662008-03-05 00:59:57 +0000399 } else if (curr->killsRegister(b)) {
Owen Anderson87a702b2007-12-16 05:44:27 +0000400 kill = curr;
401 }
402 }
403 }
404
405 return interference;
406}
407
Owen Andersondc4d6552008-01-10 00:47:01 +0000408/// processBlock - Determine how to break up PHIs in the current block. Each
409/// PHI is broken up by some combination of renaming its operands and inserting
410/// copies. This method is responsible for determining which operands receive
411/// which treatment.
Owen Anderson60a877d2007-11-07 05:17:15 +0000412void StrongPHIElimination::processBlock(MachineBasicBlock* MBB) {
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000413 LiveIntervals& LI = getAnalysis<LiveIntervals>();
Owen Andersonddd060f2008-01-10 01:36:43 +0000414 MachineRegisterInfo& MRI = MBB->getParent()->getRegInfo();
Owen Anderson60a877d2007-11-07 05:17:15 +0000415
416 // Holds names that have been added to a set in any PHI within this block
417 // before the current one.
418 std::set<unsigned> ProcessedNames;
419
Owen Andersondc4d6552008-01-10 00:47:01 +0000420 // Iterate over all the PHI nodes in this block
Owen Anderson60a877d2007-11-07 05:17:15 +0000421 MachineBasicBlock::iterator P = MBB->begin();
Owen Anderson78216bc2008-04-01 18:05:08 +0000422 while (P != MBB->end() && P->getOpcode() == TargetInstrInfo::PHI) {
Owen Andersonafc6de02007-12-10 08:07:09 +0000423 unsigned DestReg = P->getOperand(0).getReg();
Owen Anderson9860b712008-08-06 20:29:20 +0000424
Owen Andersond382f8a2008-03-26 03:03:23 +0000425 // Don't both doing PHI elimination for dead PHI's.
426 if (P->registerDefIsDead(DestReg)) {
427 ++P;
428 continue;
429 }
430
Owen Andersondfd07ea2008-03-12 04:22:57 +0000431 LiveInterval& PI = LI.getOrCreateInterval(DestReg);
Lang Hames233a60e2009-11-03 23:52:08 +0000432 SlotIndex pIdx = LI.getInstructionIndex(P).getDefIndex();
Owen Andersondfd07ea2008-03-12 04:22:57 +0000433 VNInfo* PVN = PI.getLiveRangeContaining(pIdx)->valno;
434 PhiValueNumber.insert(std::make_pair(DestReg, PVN->id));
435
Owen Andersondc4d6552008-01-10 00:47:01 +0000436 // PHIUnion is the set of incoming registers to the PHI node that
437 // are going to be renames rather than having copies inserted. This set
438 // is refinded over the course of this function. UnionedBlocks is the set
439 // of corresponding MBBs.
Owen Andersonc12417e2008-07-24 17:12:16 +0000440 std::map<unsigned, MachineBasicBlock*> PHIUnion;
Owen Andersone46611e2008-03-24 04:11:27 +0000441 SmallPtrSet<MachineBasicBlock*, 8> UnionedBlocks;
Owen Anderson60a877d2007-11-07 05:17:15 +0000442
Owen Andersondc4d6552008-01-10 00:47:01 +0000443 // Iterate over the operands of the PHI node
Owen Anderson60a877d2007-11-07 05:17:15 +0000444 for (int i = P->getNumOperands() - 1; i >= 2; i-=2) {
445 unsigned SrcReg = P->getOperand(i-1).getReg();
Owen Anderson9860b712008-08-06 20:29:20 +0000446
447 // Don't need to try to coalesce a register with itself.
448 if (SrcReg == DestReg) {
449 ProcessedNames.insert(SrcReg);
450 continue;
451 }
Owen Anderson082d3512008-09-26 18:50:46 +0000452
453 // We don't need to insert copies for implicit_defs.
454 MachineInstr* DefMI = MRI.getVRegDef(SrcReg);
455 if (DefMI->getOpcode() == TargetInstrInfo::IMPLICIT_DEF)
456 ProcessedNames.insert(SrcReg);
Owen Anderson60a877d2007-11-07 05:17:15 +0000457
Owen Andersondc4d6552008-01-10 00:47:01 +0000458 // Check for trivial interferences via liveness information, allowing us
459 // to avoid extra work later. Any registers that interfere cannot both
460 // be in the renaming set, so choose one and add copies for it instead.
461 // The conditions are:
462 // 1) if the operand is live into the PHI node's block OR
463 // 2) if the PHI node is live out of the operand's defining block OR
464 // 3) if the operand is itself a PHI node and the original PHI is
465 // live into the operand's defining block OR
466 // 4) if the operand is already being renamed for another PHI node
467 // in this block OR
468 // 5) if any two operands are defined in the same block, insert copies
469 // for one of them
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000470 if (isLiveIn(SrcReg, P->getParent(), LI) ||
Owen Andersonddd060f2008-01-10 01:36:43 +0000471 isLiveOut(P->getOperand(0).getReg(),
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000472 MRI.getVRegDef(SrcReg)->getParent(), LI) ||
Owen Andersonddd060f2008-01-10 01:36:43 +0000473 ( MRI.getVRegDef(SrcReg)->getOpcode() == TargetInstrInfo::PHI &&
474 isLiveIn(P->getOperand(0).getReg(),
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000475 MRI.getVRegDef(SrcReg)->getParent(), LI) ) ||
Owen Andersonafc6de02007-12-10 08:07:09 +0000476 ProcessedNames.count(SrcReg) ||
Owen Andersonddd060f2008-01-10 01:36:43 +0000477 UnionedBlocks.count(MRI.getVRegDef(SrcReg)->getParent())) {
Owen Andersonafc6de02007-12-10 08:07:09 +0000478
Owen Andersondc4d6552008-01-10 00:47:01 +0000479 // Add a copy for the selected register
Chris Lattner8aa797a2007-12-30 23:10:15 +0000480 MachineBasicBlock* From = P->getOperand(i).getMBB();
Owen Andersonefbcebc2007-12-23 15:37:26 +0000481 Waiting[From].insert(std::make_pair(SrcReg, DestReg));
482 UsedByAnother.insert(SrcReg);
Owen Anderson60a877d2007-11-07 05:17:15 +0000483 } else {
Owen Andersondc4d6552008-01-10 00:47:01 +0000484 // Otherwise, add it to the renaming set
Owen Andersonc12417e2008-07-24 17:12:16 +0000485 PHIUnion.insert(std::make_pair(SrcReg,P->getOperand(i).getMBB()));
Owen Andersonddd060f2008-01-10 01:36:43 +0000486 UnionedBlocks.insert(MRI.getVRegDef(SrcReg)->getParent());
Owen Anderson60a877d2007-11-07 05:17:15 +0000487 }
Owen Anderson60a877d2007-11-07 05:17:15 +0000488 }
489
Owen Andersondc4d6552008-01-10 00:47:01 +0000490 // Compute the dominator forest for the renaming set. This is a forest
491 // where the nodes are the registers and the edges represent dominance
492 // relations between the defining blocks of the registers
Owen Anderson42f9e962007-11-13 20:13:24 +0000493 std::vector<StrongPHIElimination::DomForestNode*> DF =
Owen Andersonddd060f2008-01-10 01:36:43 +0000494 computeDomForest(PHIUnion, MRI);
Owen Anderson42f9e962007-11-13 20:13:24 +0000495
Owen Andersondc4d6552008-01-10 00:47:01 +0000496 // Walk DomForest to resolve interferences at an inter-block level. This
497 // will remove registers from the renaming set (and insert copies for them)
498 // if interferences are found.
Owen Anderson62d67dd2007-12-13 05:53:03 +0000499 std::vector<std::pair<unsigned, unsigned> > localInterferences;
500 processPHIUnion(P, PHIUnion, DF, localInterferences);
501
Owen Anderson52b17332008-04-02 03:00:13 +0000502 // If one of the inputs is defined in the same block as the current PHI
503 // then we need to check for a local interference between that input and
504 // the PHI.
Owen Andersonc12417e2008-07-24 17:12:16 +0000505 for (std::map<unsigned, MachineBasicBlock*>::iterator I = PHIUnion.begin(),
Owen Anderson52b17332008-04-02 03:00:13 +0000506 E = PHIUnion.end(); I != E; ++I)
507 if (MRI.getVRegDef(I->first)->getParent() == P->getParent())
508 localInterferences.push_back(std::make_pair(I->first,
509 P->getOperand(0).getReg()));
510
Owen Andersondc4d6552008-01-10 00:47:01 +0000511 // The dominator forest walk may have returned some register pairs whose
Owen Anderson52b17332008-04-02 03:00:13 +0000512 // interference cannot be determined from dominator analysis. We now
Owen Andersondc4d6552008-01-10 00:47:01 +0000513 // examine these pairs for local interferences.
Owen Anderson87a702b2007-12-16 05:44:27 +0000514 for (std::vector<std::pair<unsigned, unsigned> >::iterator I =
515 localInterferences.begin(), E = localInterferences.end(); I != E; ++I) {
516 std::pair<unsigned, unsigned> p = *I;
517
Owen Anderson87a702b2007-12-16 05:44:27 +0000518 MachineDominatorTree& MDT = getAnalysis<MachineDominatorTree>();
519
520 // Determine the block we need to scan and the relationship between
521 // the two registers
522 MachineBasicBlock* scan = 0;
523 unsigned mode = 0;
Owen Andersonddd060f2008-01-10 01:36:43 +0000524 if (MRI.getVRegDef(p.first)->getParent() ==
525 MRI.getVRegDef(p.second)->getParent()) {
526 scan = MRI.getVRegDef(p.first)->getParent();
Owen Anderson87a702b2007-12-16 05:44:27 +0000527 mode = 0; // Same block
Owen Andersonddd060f2008-01-10 01:36:43 +0000528 } else if (MDT.dominates(MRI.getVRegDef(p.first)->getParent(),
529 MRI.getVRegDef(p.second)->getParent())) {
530 scan = MRI.getVRegDef(p.second)->getParent();
Owen Anderson87a702b2007-12-16 05:44:27 +0000531 mode = 1; // First dominates second
532 } else {
Owen Andersonddd060f2008-01-10 01:36:43 +0000533 scan = MRI.getVRegDef(p.first)->getParent();
Owen Anderson87a702b2007-12-16 05:44:27 +0000534 mode = 2; // Second dominates first
535 }
536
537 // If there's an interference, we need to insert copies
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000538 if (interferes(p.first, p.second, scan, LI, mode)) {
Owen Anderson87a702b2007-12-16 05:44:27 +0000539 // Insert copies for First
540 for (int i = P->getNumOperands() - 1; i >= 2; i-=2) {
541 if (P->getOperand(i-1).getReg() == p.first) {
542 unsigned SrcReg = p.first;
543 MachineBasicBlock* From = P->getOperand(i).getMBB();
544
Owen Andersonefbcebc2007-12-23 15:37:26 +0000545 Waiting[From].insert(std::make_pair(SrcReg,
546 P->getOperand(0).getReg()));
547 UsedByAnother.insert(SrcReg);
548
Owen Anderson87a702b2007-12-16 05:44:27 +0000549 PHIUnion.erase(SrcReg);
550 }
551 }
552 }
553 }
Owen Anderson42f9e962007-11-13 20:13:24 +0000554
Owen Anderson52b17332008-04-02 03:00:13 +0000555 // Add the renaming set for this PHI node to our overall renaming information
Owen Andersone04cec02008-09-22 21:58:58 +0000556 for (std::map<unsigned, MachineBasicBlock*>::iterator QI = PHIUnion.begin(),
557 QE = PHIUnion.end(); QI != QE; ++QI) {
Chris Lattnerbbbfa992009-08-23 06:35:02 +0000558 DEBUG(errs() << "Adding Renaming: " << QI->first << " -> "
559 << P->getOperand(0).getReg() << "\n");
Owen Andersone04cec02008-09-22 21:58:58 +0000560 }
561
Owen Anderson0c5714b2008-01-08 21:54:52 +0000562 RenameSets.insert(std::make_pair(P->getOperand(0).getReg(), PHIUnion));
Owen Andersoncae8d8d2007-12-22 04:59:10 +0000563
Owen Andersondc4d6552008-01-10 00:47:01 +0000564 // Remember which registers are already renamed, so that we don't try to
565 // rename them for another PHI node in this block
Owen Andersonc12417e2008-07-24 17:12:16 +0000566 for (std::map<unsigned, MachineBasicBlock*>::iterator I = PHIUnion.begin(),
Owen Anderson00316712008-03-12 03:13:29 +0000567 E = PHIUnion.end(); I != E; ++I)
568 ProcessedNames.insert(I->first);
Owen Andersondc4d6552008-01-10 00:47:01 +0000569
Owen Anderson60a877d2007-11-07 05:17:15 +0000570 ++P;
571 }
Owen Andersonee49b532007-11-06 05:22:43 +0000572}
573
Gabor Greif2cf36e02008-03-06 10:51:21 +0000574/// processPHIUnion - Take a set of candidate registers to be coalesced when
Owen Anderson965b4672007-12-16 04:07:23 +0000575/// decomposing the PHI instruction. Use the DominanceForest to remove the ones
576/// that are known to interfere, and flag others that need to be checked for
577/// local interferences.
Owen Andersond525f662007-12-11 20:12:11 +0000578void StrongPHIElimination::processPHIUnion(MachineInstr* Inst,
Owen Andersonc12417e2008-07-24 17:12:16 +0000579 std::map<unsigned, MachineBasicBlock*>& PHIUnion,
Owen Anderson62d67dd2007-12-13 05:53:03 +0000580 std::vector<StrongPHIElimination::DomForestNode*>& DF,
581 std::vector<std::pair<unsigned, unsigned> >& locals) {
Owen Andersond525f662007-12-11 20:12:11 +0000582
583 std::vector<DomForestNode*> worklist(DF.begin(), DF.end());
584 SmallPtrSet<DomForestNode*, 4> visited;
585
Owen Andersonddd060f2008-01-10 01:36:43 +0000586 // Code is still in SSA form, so we can use MRI::getVRegDef()
587 MachineRegisterInfo& MRI = Inst->getParent()->getParent()->getRegInfo();
588
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000589 LiveIntervals& LI = getAnalysis<LiveIntervals>();
Owen Andersond525f662007-12-11 20:12:11 +0000590 unsigned DestReg = Inst->getOperand(0).getReg();
591
Owen Anderson965b4672007-12-16 04:07:23 +0000592 // DF walk on the DomForest
Owen Andersond525f662007-12-11 20:12:11 +0000593 while (!worklist.empty()) {
594 DomForestNode* DFNode = worklist.back();
595
Owen Andersond525f662007-12-11 20:12:11 +0000596 visited.insert(DFNode);
597
598 bool inserted = false;
Owen Andersond525f662007-12-11 20:12:11 +0000599 for (DomForestNode::iterator CI = DFNode->begin(), CE = DFNode->end();
600 CI != CE; ++CI) {
601 DomForestNode* child = *CI;
Owen Anderson3b489522008-01-21 22:01:01 +0000602
603 // If the current node is live-out of the defining block of one of its
Owen Andersona6b19262008-01-21 22:03:00 +0000604 // children, insert a copy for it. NOTE: The paper actually calls for
605 // a more elaborate heuristic for determining whether to insert copies
606 // for the child or the parent. In the interest of simplicity, we're
607 // just always choosing the parent.
Owen Andersonddd060f2008-01-10 01:36:43 +0000608 if (isLiveOut(DFNode->getReg(),
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000609 MRI.getVRegDef(child->getReg())->getParent(), LI)) {
Owen Andersond525f662007-12-11 20:12:11 +0000610 // Insert copies for parent
611 for (int i = Inst->getNumOperands() - 1; i >= 2; i-=2) {
612 if (Inst->getOperand(i-1).getReg() == DFNode->getReg()) {
Owen Andersoned2ffa22007-12-12 01:25:08 +0000613 unsigned SrcReg = DFNode->getReg();
Owen Andersond525f662007-12-11 20:12:11 +0000614 MachineBasicBlock* From = Inst->getOperand(i).getMBB();
615
Owen Andersonefbcebc2007-12-23 15:37:26 +0000616 Waiting[From].insert(std::make_pair(SrcReg, DestReg));
617 UsedByAnother.insert(SrcReg);
618
Owen Andersoned2ffa22007-12-12 01:25:08 +0000619 PHIUnion.erase(SrcReg);
Owen Andersond525f662007-12-11 20:12:11 +0000620 }
621 }
Owen Anderson3b489522008-01-21 22:01:01 +0000622
623 // If a node is live-in to the defining block of one of its children, but
624 // not live-out, then we need to scan that block for local interferences.
Owen Andersonddd060f2008-01-10 01:36:43 +0000625 } else if (isLiveIn(DFNode->getReg(),
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000626 MRI.getVRegDef(child->getReg())->getParent(), LI) ||
Owen Andersonddd060f2008-01-10 01:36:43 +0000627 MRI.getVRegDef(DFNode->getReg())->getParent() ==
628 MRI.getVRegDef(child->getReg())->getParent()) {
Owen Anderson62d67dd2007-12-13 05:53:03 +0000629 // Add (p, c) to possible local interferences
630 locals.push_back(std::make_pair(DFNode->getReg(), child->getReg()));
Owen Andersond525f662007-12-11 20:12:11 +0000631 }
Owen Anderson965b4672007-12-16 04:07:23 +0000632
Owen Anderson4ba08ec2007-12-13 05:43:37 +0000633 if (!visited.count(child)) {
634 worklist.push_back(child);
635 inserted = true;
Owen Andersond525f662007-12-11 20:12:11 +0000636 }
637 }
638
639 if (!inserted) worklist.pop_back();
640 }
641}
642
Owen Andersonefbcebc2007-12-23 15:37:26 +0000643/// ScheduleCopies - Insert copies into predecessor blocks, scheduling
644/// them properly so as to avoid the 'lost copy' and the 'virtual swap'
645/// problems.
646///
647/// Based on "Practical Improvements to the Construction and Destruction
648/// of Static Single Assignment Form" by Briggs, et al.
Owen Andersonf1519e82007-12-24 22:12:23 +0000649void StrongPHIElimination::ScheduleCopies(MachineBasicBlock* MBB,
650 std::set<unsigned>& pushed) {
Owen Anderson38b42502008-06-04 17:55:58 +0000651 // FIXME: This function needs to update LiveIntervals
Owen Anderson8481ee72008-10-02 19:40:33 +0000652 std::multimap<unsigned, unsigned>& copy_set= Waiting[MBB];
Owen Andersonefbcebc2007-12-23 15:37:26 +0000653
Owen Anderson8481ee72008-10-02 19:40:33 +0000654 std::multimap<unsigned, unsigned> worklist;
Owen Andersonefbcebc2007-12-23 15:37:26 +0000655 std::map<unsigned, unsigned> map;
656
657 // Setup worklist of initial copies
Argyrios Kyrtzidis0304b872008-10-04 08:11:49 +0000658 for (std::multimap<unsigned, unsigned>::iterator I = copy_set.begin(),
Owen Andersonefbcebc2007-12-23 15:37:26 +0000659 E = copy_set.end(); I != E; ) {
660 map.insert(std::make_pair(I->first, I->first));
661 map.insert(std::make_pair(I->second, I->second));
662
Owen Andersonc7c00362008-03-29 01:58:47 +0000663 if (!UsedByAnother.count(I->second)) {
Owen Andersonefbcebc2007-12-23 15:37:26 +0000664 worklist.insert(*I);
665
666 // Avoid iterator invalidation
Owen Anderson8481ee72008-10-02 19:40:33 +0000667 std::multimap<unsigned, unsigned>::iterator OI = I;
Owen Andersonefbcebc2007-12-23 15:37:26 +0000668 ++I;
Owen Anderson8481ee72008-10-02 19:40:33 +0000669 copy_set.erase(OI);
Owen Andersonefbcebc2007-12-23 15:37:26 +0000670 } else {
671 ++I;
672 }
673 }
674
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000675 LiveIntervals& LI = getAnalysis<LiveIntervals>();
Owen Anderson0d893b42008-01-08 05:16:15 +0000676 MachineFunction* MF = MBB->getParent();
Owen Andersonddd060f2008-01-10 01:36:43 +0000677 MachineRegisterInfo& MRI = MF->getRegInfo();
Owen Anderson0d893b42008-01-08 05:16:15 +0000678 const TargetInstrInfo *TII = MF->getTarget().getInstrInfo();
Owen Andersonefbcebc2007-12-23 15:37:26 +0000679
Owen Anderson38b42502008-06-04 17:55:58 +0000680 SmallVector<std::pair<unsigned, MachineInstr*>, 4> InsertedPHIDests;
681
Owen Andersonefbcebc2007-12-23 15:37:26 +0000682 // Iterate over the worklist, inserting copies
683 while (!worklist.empty() || !copy_set.empty()) {
684 while (!worklist.empty()) {
Owen Anderson8481ee72008-10-02 19:40:33 +0000685 std::multimap<unsigned, unsigned>::iterator WI = worklist.begin();
686 std::pair<unsigned, unsigned> curr = *WI;
687 worklist.erase(WI);
Owen Andersonefbcebc2007-12-23 15:37:26 +0000688
Owen Anderson0d893b42008-01-08 05:16:15 +0000689 const TargetRegisterClass *RC = MF->getRegInfo().getRegClass(curr.first);
690
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000691 if (isLiveOut(curr.second, MBB, LI)) {
Owen Anderson0d893b42008-01-08 05:16:15 +0000692 // Create a temporary
693 unsigned t = MF->getRegInfo().createVirtualRegister(RC);
694
695 // Insert copy from curr.second to a temporary at
696 // the Phi defining curr.second
Owen Andersonddd060f2008-01-10 01:36:43 +0000697 MachineBasicBlock::iterator PI = MRI.getVRegDef(curr.second);
698 TII->copyRegToReg(*PI->getParent(), PI, t,
Owen Anderson0d893b42008-01-08 05:16:15 +0000699 curr.second, RC, RC);
700
Chris Lattnerbbbfa992009-08-23 06:35:02 +0000701 DEBUG(errs() << "Inserted copy from " << curr.second << " to " << t
702 << "\n");
Owen Anderson8481ee72008-10-02 19:40:33 +0000703
Owen Andersonefbcebc2007-12-23 15:37:26 +0000704 // Push temporary on Stacks
Owen Anderson0d893b42008-01-08 05:16:15 +0000705 Stacks[curr.second].push_back(t);
706
707 // Insert curr.second in pushed
708 pushed.insert(curr.second);
Owen Anderson724acf12008-08-07 18:28:07 +0000709
710 // Create a live interval for this temporary
711 InsertedPHIDests.push_back(std::make_pair(t, --PI));
Owen Andersonefbcebc2007-12-23 15:37:26 +0000712 }
713
714 // Insert copy from map[curr.first] to curr.second
Owen Anderson9c2efa82008-01-10 00:01:41 +0000715 TII->copyRegToReg(*MBB, MBB->getFirstTerminator(), curr.second,
Owen Anderson0d893b42008-01-08 05:16:15 +0000716 map[curr.first], RC, RC);
Owen Andersonefbcebc2007-12-23 15:37:26 +0000717 map[curr.first] = curr.second;
Chris Lattnerbbbfa992009-08-23 06:35:02 +0000718 DEBUG(errs() << "Inserted copy from " << curr.first << " to "
719 << curr.second << "\n");
Owen Andersonefbcebc2007-12-23 15:37:26 +0000720
Owen Anderson38b42502008-06-04 17:55:58 +0000721 // Push this copy onto InsertedPHICopies so we can
722 // update LiveIntervals with it.
723 MachineBasicBlock::iterator MI = MBB->getFirstTerminator();
724 InsertedPHIDests.push_back(std::make_pair(curr.second, --MI));
725
Owen Andersonefbcebc2007-12-23 15:37:26 +0000726 // If curr.first is a destination in copy_set...
Owen Anderson8481ee72008-10-02 19:40:33 +0000727 for (std::multimap<unsigned, unsigned>::iterator I = copy_set.begin(),
Owen Andersonefbcebc2007-12-23 15:37:26 +0000728 E = copy_set.end(); I != E; )
729 if (curr.first == I->second) {
730 std::pair<unsigned, unsigned> temp = *I;
Owen Anderson8481ee72008-10-02 19:40:33 +0000731 worklist.insert(temp);
Owen Andersonefbcebc2007-12-23 15:37:26 +0000732
733 // Avoid iterator invalidation
Owen Anderson8481ee72008-10-02 19:40:33 +0000734 std::multimap<unsigned, unsigned>::iterator OI = I;
Owen Andersonefbcebc2007-12-23 15:37:26 +0000735 ++I;
Owen Anderson8481ee72008-10-02 19:40:33 +0000736 copy_set.erase(OI);
Owen Andersonefbcebc2007-12-23 15:37:26 +0000737
738 break;
739 } else {
740 ++I;
741 }
742 }
743
744 if (!copy_set.empty()) {
Owen Anderson8481ee72008-10-02 19:40:33 +0000745 std::multimap<unsigned, unsigned>::iterator CI = copy_set.begin();
746 std::pair<unsigned, unsigned> curr = *CI;
Owen Andersonefbcebc2007-12-23 15:37:26 +0000747 worklist.insert(curr);
Owen Anderson8481ee72008-10-02 19:40:33 +0000748 copy_set.erase(CI);
Owen Anderson27d66812008-08-13 17:25:42 +0000749
750 LiveInterval& I = LI.getInterval(curr.second);
751 MachineBasicBlock::iterator term = MBB->getFirstTerminator();
Lang Hames233a60e2009-11-03 23:52:08 +0000752 SlotIndex endIdx = SlotIndex();
Owen Anderson27d66812008-08-13 17:25:42 +0000753 if (term != MBB->end())
754 endIdx = LI.getInstructionIndex(term);
755 else
756 endIdx = LI.getMBBEndIdx(MBB);
757
758 if (I.liveAt(endIdx)) {
759 const TargetRegisterClass *RC =
760 MF->getRegInfo().getRegClass(curr.first);
761
762 // Insert a copy from dest to a new temporary t at the end of b
763 unsigned t = MF->getRegInfo().createVirtualRegister(RC);
764 TII->copyRegToReg(*MBB, MBB->getFirstTerminator(), t,
765 curr.second, RC, RC);
766 map[curr.second] = t;
767
768 MachineBasicBlock::iterator TI = MBB->getFirstTerminator();
769 InsertedPHIDests.push_back(std::make_pair(t, --TI));
770 }
Owen Andersonefbcebc2007-12-23 15:37:26 +0000771 }
772 }
Owen Anderson38b42502008-06-04 17:55:58 +0000773
774 // Renumber the instructions so that we can perform the index computations
775 // needed to create new live intervals.
Lang Hames233a60e2009-11-03 23:52:08 +0000776 LI.renumber();
Owen Anderson38b42502008-06-04 17:55:58 +0000777
778 // For copies that we inserted at the ends of predecessors, we construct
779 // live intervals. This is pretty easy, since we know that the destination
780 // register cannot have be in live at that point previously. We just have
781 // to make sure that, for registers that serve as inputs to more than one
782 // PHI, we don't create multiple overlapping live intervals.
783 std::set<unsigned> RegHandled;
784 for (SmallVector<std::pair<unsigned, MachineInstr*>, 4>::iterator I =
Owen Anderson27d66812008-08-13 17:25:42 +0000785 InsertedPHIDests.begin(), E = InsertedPHIDests.end(); I != E; ++I) {
Owen Andersone04cec02008-09-22 21:58:58 +0000786 if (RegHandled.insert(I->first).second) {
787 LiveInterval& Int = LI.getOrCreateInterval(I->first);
Lang Hames233a60e2009-11-03 23:52:08 +0000788 SlotIndex instrIdx = LI.getInstructionIndex(I->second);
789 if (Int.liveAt(instrIdx.getDefIndex()))
790 Int.removeRange(instrIdx.getDefIndex(),
791 LI.getMBBEndIdx(I->second->getParent()).getNextSlot(),
Owen Andersone04cec02008-09-22 21:58:58 +0000792 true);
793
794 LiveRange R = LI.addLiveRangeToEndOfBlock(I->first, I->second);
Lang Hames52c1afc2009-08-10 23:43:28 +0000795 R.valno->setCopy(I->second);
Lang Hames233a60e2009-11-03 23:52:08 +0000796 R.valno->def = LI.getInstructionIndex(I->second).getDefIndex();
Owen Andersone04cec02008-09-22 21:58:58 +0000797 }
Owen Anderson27d66812008-08-13 17:25:42 +0000798 }
Owen Andersonefbcebc2007-12-23 15:37:26 +0000799}
800
Owen Andersonf1519e82007-12-24 22:12:23 +0000801/// InsertCopies - insert copies into MBB and all of its successors
Owen Anderson724acf12008-08-07 18:28:07 +0000802void StrongPHIElimination::InsertCopies(MachineDomTreeNode* MDTN,
Owen Andersone46611e2008-03-24 04:11:27 +0000803 SmallPtrSet<MachineBasicBlock*, 16>& visited) {
Owen Anderson724acf12008-08-07 18:28:07 +0000804 MachineBasicBlock* MBB = MDTN->getBlock();
Owen Anderson719fef62008-01-09 10:32:30 +0000805 visited.insert(MBB);
806
Owen Andersonf1519e82007-12-24 22:12:23 +0000807 std::set<unsigned> pushed;
808
Owen Anderson724acf12008-08-07 18:28:07 +0000809 LiveIntervals& LI = getAnalysis<LiveIntervals>();
Owen Andersonf1519e82007-12-24 22:12:23 +0000810 // Rewrite register uses from Stacks
811 for (MachineBasicBlock::iterator I = MBB->begin(), E = MBB->end();
Owen Anderson724acf12008-08-07 18:28:07 +0000812 I != E; ++I) {
813 if (I->getOpcode() == TargetInstrInfo::PHI)
814 continue;
815
Owen Andersonf1519e82007-12-24 22:12:23 +0000816 for (unsigned i = 0; i < I->getNumOperands(); ++i)
Dan Gohmand735b802008-10-03 15:45:36 +0000817 if (I->getOperand(i).isReg() &&
Owen Andersonf1519e82007-12-24 22:12:23 +0000818 Stacks[I->getOperand(i).getReg()].size()) {
Owen Anderson724acf12008-08-07 18:28:07 +0000819 // Remove the live range for the old vreg.
820 LiveInterval& OldInt = LI.getInterval(I->getOperand(i).getReg());
Lang Hames233a60e2009-11-03 23:52:08 +0000821 LiveInterval::iterator OldLR =
822 OldInt.FindLiveRangeContaining(LI.getInstructionIndex(I).getUseIndex());
Owen Anderson724acf12008-08-07 18:28:07 +0000823 if (OldLR != OldInt.end())
824 OldInt.removeRange(*OldLR, true);
825
826 // Change the register
Owen Andersonf1519e82007-12-24 22:12:23 +0000827 I->getOperand(i).setReg(Stacks[I->getOperand(i).getReg()].back());
Owen Anderson724acf12008-08-07 18:28:07 +0000828
829 // Add a live range for the new vreg
830 LiveInterval& Int = LI.getInterval(I->getOperand(i).getReg());
831 VNInfo* FirstVN = *Int.vni_begin();
Lang Hames857c4e02009-06-17 21:01:20 +0000832 FirstVN->setHasPHIKill(false);
Owen Anderson724acf12008-08-07 18:28:07 +0000833 if (I->getOperand(i).isKill())
Lang Hames233a60e2009-11-03 23:52:08 +0000834 FirstVN->addKill(LI.getInstructionIndex(I).getUseIndex());
Owen Anderson724acf12008-08-07 18:28:07 +0000835
836 LiveRange LR (LI.getMBBStartIdx(I->getParent()),
Lang Hames233a60e2009-11-03 23:52:08 +0000837 LI.getInstructionIndex(I).getUseIndex().getNextSlot(),
Owen Anderson724acf12008-08-07 18:28:07 +0000838 FirstVN);
839
840 Int.addRange(LR);
Owen Andersonf1519e82007-12-24 22:12:23 +0000841 }
Owen Anderson724acf12008-08-07 18:28:07 +0000842 }
Owen Andersonf1519e82007-12-24 22:12:23 +0000843
844 // Schedule the copies for this block
845 ScheduleCopies(MBB, pushed);
846
Owen Anderson724acf12008-08-07 18:28:07 +0000847 // Recur down the dominator tree.
848 for (MachineDomTreeNode::iterator I = MDTN->begin(),
849 E = MDTN->end(); I != E; ++I)
850 if (!visited.count((*I)->getBlock()))
Owen Anderson719fef62008-01-09 10:32:30 +0000851 InsertCopies(*I, visited);
Owen Andersonf1519e82007-12-24 22:12:23 +0000852
853 // As we exit this block, pop the names we pushed while processing it
854 for (std::set<unsigned>::iterator I = pushed.begin(),
855 E = pushed.end(); I != E; ++I)
856 Stacks[*I].pop_back();
857}
858
Owen Andersonfae86ed2008-09-23 04:37:10 +0000859bool StrongPHIElimination::mergeLiveIntervals(unsigned primary,
Owen Anderson27d66812008-08-13 17:25:42 +0000860 unsigned secondary) {
Owen Anderson755ebab2008-03-17 06:08:26 +0000861
862 LiveIntervals& LI = getAnalysis<LiveIntervals>();
863 LiveInterval& LHS = LI.getOrCreateInterval(primary);
864 LiveInterval& RHS = LI.getOrCreateInterval(secondary);
865
Lang Hames233a60e2009-11-03 23:52:08 +0000866 LI.renumber();
Owen Andersone04cec02008-09-22 21:58:58 +0000867
Owen Andersonb9fb8d12008-07-30 00:21:16 +0000868 DenseMap<VNInfo*, VNInfo*> VNMap;
Owen Andersone04cec02008-09-22 21:58:58 +0000869 for (LiveInterval::iterator I = RHS.begin(), E = RHS.end(); I != E; ++I) {
870 LiveRange R = *I;
871
Lang Hames233a60e2009-11-03 23:52:08 +0000872 SlotIndex Start = R.start;
873 SlotIndex End = R.end;
Owen Andersonfae86ed2008-09-23 04:37:10 +0000874 if (LHS.getLiveRangeContaining(Start))
875 return false;
Owen Andersone04cec02008-09-22 21:58:58 +0000876
Owen Andersonfae86ed2008-09-23 04:37:10 +0000877 if (LHS.getLiveRangeContaining(End))
878 return false;
Owen Andersone04cec02008-09-22 21:58:58 +0000879
880 LiveInterval::iterator RI = std::upper_bound(LHS.begin(), LHS.end(), R);
881 if (RI != LHS.end() && RI->start < End)
Owen Andersonfae86ed2008-09-23 04:37:10 +0000882 return false;
883 }
884
885 for (LiveInterval::iterator I = RHS.begin(), E = RHS.end(); I != E; ++I) {
886 LiveRange R = *I;
887 VNInfo* OldVN = R.valno;
888 VNInfo*& NewVN = VNMap[OldVN];
889 if (!NewVN) {
Lang Hames857c4e02009-06-17 21:01:20 +0000890 NewVN = LHS.createValueCopy(OldVN, LI.getVNInfoAllocator());
Owen Andersone04cec02008-09-22 21:58:58 +0000891 }
Owen Andersonfae86ed2008-09-23 04:37:10 +0000892
893 LiveRange LR (R.start, R.end, NewVN);
894 LHS.addRange(LR);
Owen Andersonb9fb8d12008-07-30 00:21:16 +0000895 }
Owen Anderson55c64352008-07-25 21:08:41 +0000896
Owen Andersone04cec02008-09-22 21:58:58 +0000897 LI.removeInterval(RHS.reg);
Owen Andersonfae86ed2008-09-23 04:37:10 +0000898
899 return true;
Owen Anderson00316712008-03-12 03:13:29 +0000900}
901
Owen Andersona4ad2e72007-11-06 04:49:43 +0000902bool StrongPHIElimination::runOnMachineFunction(MachineFunction &Fn) {
Owen Andersonc7c00362008-03-29 01:58:47 +0000903 LiveIntervals& LI = getAnalysis<LiveIntervals>();
904
Owen Andersonefbcebc2007-12-23 15:37:26 +0000905 // Compute DFS numbers of each block
Owen Andersona4ad2e72007-11-06 04:49:43 +0000906 computeDFS(Fn);
907
Owen Andersonefbcebc2007-12-23 15:37:26 +0000908 // Determine which phi node operands need copies
Owen Anderson60a877d2007-11-07 05:17:15 +0000909 for (MachineFunction::iterator I = Fn.begin(), E = Fn.end(); I != E; ++I)
910 if (!I->empty() &&
911 I->begin()->getOpcode() == TargetInstrInfo::PHI)
912 processBlock(I);
Owen Andersona4ad2e72007-11-06 04:49:43 +0000913
Owen Anderson883771f2008-08-06 22:08:58 +0000914 // Break interferences where two different phis want to coalesce
915 // in the same register.
916 std::set<unsigned> seen;
917 typedef std::map<unsigned, std::map<unsigned, MachineBasicBlock*> >
918 RenameSetType;
919 for (RenameSetType::iterator I = RenameSets.begin(), E = RenameSets.end();
920 I != E; ++I) {
921 for (std::map<unsigned, MachineBasicBlock*>::iterator
922 OI = I->second.begin(), OE = I->second.end(); OI != OE; ) {
923 if (!seen.count(OI->first)) {
924 seen.insert(OI->first);
925 ++OI;
926 } else {
927 Waiting[OI->second].insert(std::make_pair(OI->first, I->first));
928 unsigned reg = OI->first;
929 ++OI;
930 I->second.erase(reg);
Chris Lattnerbbbfa992009-08-23 06:35:02 +0000931 DEBUG(errs() << "Removing Renaming: " << reg << " -> " << I->first
932 << "\n");
Owen Anderson883771f2008-08-06 22:08:58 +0000933 }
934 }
935 }
936
Owen Andersonefbcebc2007-12-23 15:37:26 +0000937 // Insert copies
Owen Andersona9efb262008-06-05 17:22:53 +0000938 // FIXME: This process should probably preserve LiveIntervals
Owen Andersone46611e2008-03-24 04:11:27 +0000939 SmallPtrSet<MachineBasicBlock*, 16> visited;
Owen Anderson724acf12008-08-07 18:28:07 +0000940 MachineDominatorTree& MDT = getAnalysis<MachineDominatorTree>();
941 InsertCopies(MDT.getRootNode(), visited);
Owen Andersonefbcebc2007-12-23 15:37:26 +0000942
Owen Anderson0c5714b2008-01-08 21:54:52 +0000943 // Perform renaming
Owen Anderson0c5714b2008-01-08 21:54:52 +0000944 for (RenameSetType::iterator I = RenameSets.begin(), E = RenameSets.end();
945 I != E; ++I)
Owen Anderson883771f2008-08-06 22:08:58 +0000946 while (I->second.size()) {
947 std::map<unsigned, MachineBasicBlock*>::iterator SI = I->second.begin();
948
Chris Lattnerbbbfa992009-08-23 06:35:02 +0000949 DEBUG(errs() << "Renaming: " << SI->first << " -> " << I->first << "\n");
Owen Andersone04cec02008-09-22 21:58:58 +0000950
Owen Anderson883771f2008-08-06 22:08:58 +0000951 if (SI->first != I->first) {
Owen Andersonfae86ed2008-09-23 04:37:10 +0000952 if (mergeLiveIntervals(I->first, SI->first)) {
953 Fn.getRegInfo().replaceRegWith(SI->first, I->first);
Owen Anderson883771f2008-08-06 22:08:58 +0000954
Owen Andersonfae86ed2008-09-23 04:37:10 +0000955 if (RenameSets.count(SI->first)) {
956 I->second.insert(RenameSets[SI->first].begin(),
957 RenameSets[SI->first].end());
958 RenameSets.erase(SI->first);
959 }
960 } else {
961 // Insert a last-minute copy if a conflict was detected.
962 const TargetInstrInfo *TII = Fn.getTarget().getInstrInfo();
963 const TargetRegisterClass *RC = Fn.getRegInfo().getRegClass(I->first);
964 TII->copyRegToReg(*SI->second, SI->second->getFirstTerminator(),
965 I->first, SI->first, RC, RC);
966
Lang Hames233a60e2009-11-03 23:52:08 +0000967 LI.renumber();
Owen Andersonfae86ed2008-09-23 04:37:10 +0000968
969 LiveInterval& Int = LI.getOrCreateInterval(I->first);
Lang Hames233a60e2009-11-03 23:52:08 +0000970 SlotIndex instrIdx =
Owen Andersonfae86ed2008-09-23 04:37:10 +0000971 LI.getInstructionIndex(--SI->second->getFirstTerminator());
Lang Hames233a60e2009-11-03 23:52:08 +0000972 if (Int.liveAt(instrIdx.getDefIndex()))
973 Int.removeRange(instrIdx.getDefIndex(),
974 LI.getMBBEndIdx(SI->second).getNextSlot(), true);
Owen Andersonfae86ed2008-09-23 04:37:10 +0000975
976 LiveRange R = LI.addLiveRangeToEndOfBlock(I->first,
977 --SI->second->getFirstTerminator());
Lang Hames52c1afc2009-08-10 23:43:28 +0000978 R.valno->setCopy(--SI->second->getFirstTerminator());
Lang Hames233a60e2009-11-03 23:52:08 +0000979 R.valno->def = instrIdx.getDefIndex();
Owen Andersonfae86ed2008-09-23 04:37:10 +0000980
Chris Lattnerbbbfa992009-08-23 06:35:02 +0000981 DEBUG(errs() << "Renaming failed: " << SI->first << " -> "
982 << I->first << "\n");
Owen Anderson883771f2008-08-06 22:08:58 +0000983 }
Owen Anderson883771f2008-08-06 22:08:58 +0000984 }
985
Owen Anderson05472ee2008-09-30 23:58:47 +0000986 LiveInterval& Int = LI.getOrCreateInterval(I->first);
987 const LiveRange* LR =
988 Int.getLiveRangeContaining(LI.getMBBEndIdx(SI->second));
Lang Hames857c4e02009-06-17 21:01:20 +0000989 LR->valno->setHasPHIKill(true);
Owen Anderson05472ee2008-09-30 23:58:47 +0000990
Owen Anderson883771f2008-08-06 22:08:58 +0000991 I->second.erase(SI->first);
Owen Anderson00316712008-03-12 03:13:29 +0000992 }
Owen Anderson0c5714b2008-01-08 21:54:52 +0000993
Owen Anderson0c5714b2008-01-08 21:54:52 +0000994 // Remove PHIs
Owen Anderson97ca75e2008-01-22 23:58:54 +0000995 std::vector<MachineInstr*> phis;
996 for (MachineFunction::iterator I = Fn.begin(), E = Fn.end(); I != E; ++I) {
Owen Anderson0c5714b2008-01-08 21:54:52 +0000997 for (MachineBasicBlock::iterator BI = I->begin(), BE = I->end();
998 BI != BE; ++BI)
999 if (BI->getOpcode() == TargetInstrInfo::PHI)
Owen Anderson97ca75e2008-01-22 23:58:54 +00001000 phis.push_back(BI);
1001 }
1002
1003 for (std::vector<MachineInstr*>::iterator I = phis.begin(), E = phis.end();
Owen Andersonc7c00362008-03-29 01:58:47 +00001004 I != E; ) {
1005 MachineInstr* PInstr = *(I++);
1006
Owen Anderson71ac0be2008-08-05 21:18:51 +00001007 // If this is a dead PHI node, then remove it from LiveIntervals.
1008 unsigned DestReg = PInstr->getOperand(0).getReg();
1009 LiveInterval& PI = LI.getInterval(DestReg);
1010 if (PInstr->registerDefIsDead(DestReg)) {
1011 if (PI.containsOneValue()) {
1012 LI.removeInterval(DestReg);
1013 } else {
Lang Hames233a60e2009-11-03 23:52:08 +00001014 SlotIndex idx = LI.getInstructionIndex(PInstr).getDefIndex();
Owen Anderson71ac0be2008-08-05 21:18:51 +00001015 PI.removeRange(*PI.getLiveRangeContaining(idx), true);
1016 }
1017 } else {
Owen Andersone7b82052008-08-05 20:51:26 +00001018 // Trim live intervals of input registers. They are no longer live into
Owen Anderson9b491202008-08-06 18:36:17 +00001019 // this block if they died after the PHI. If they lived after it, don't
1020 // trim them because they might have other legitimate uses.
Owen Andersone7b82052008-08-05 20:51:26 +00001021 for (unsigned i = 1; i < PInstr->getNumOperands(); i += 2) {
1022 unsigned reg = PInstr->getOperand(i).getReg();
Owen Anderson883771f2008-08-06 22:08:58 +00001023
Owen Andersone7b82052008-08-05 20:51:26 +00001024 MachineBasicBlock* MBB = PInstr->getOperand(i+1).getMBB();
1025 LiveInterval& InputI = LI.getInterval(reg);
1026 if (MBB != PInstr->getParent() &&
Owen Anderson9b491202008-08-06 18:36:17 +00001027 InputI.liveAt(LI.getMBBStartIdx(PInstr->getParent())) &&
Lang Hames233a60e2009-11-03 23:52:08 +00001028 InputI.expiredAt(LI.getInstructionIndex(PInstr).getNextIndex()))
Owen Andersone7b82052008-08-05 20:51:26 +00001029 InputI.removeRange(LI.getMBBStartIdx(PInstr->getParent()),
1030 LI.getInstructionIndex(PInstr),
1031 true);
Owen Andersond382f8a2008-03-26 03:03:23 +00001032 }
Owen Andersone7b82052008-08-05 20:51:26 +00001033
Owen Anderson71ac0be2008-08-05 21:18:51 +00001034 // If the PHI is not dead, then the valno defined by the PHI
1035 // now has an unknown def.
Lang Hames233a60e2009-11-03 23:52:08 +00001036 SlotIndex idx = LI.getInstructionIndex(PInstr).getDefIndex();
Owen Anderson71ac0be2008-08-05 21:18:51 +00001037 const LiveRange* PLR = PI.getLiveRangeContaining(idx);
Lang Hames857c4e02009-06-17 21:01:20 +00001038 PLR->valno->setIsPHIDef(true);
Owen Anderson71ac0be2008-08-05 21:18:51 +00001039 LiveRange R (LI.getMBBStartIdx(PInstr->getParent()),
1040 PLR->start, PLR->valno);
1041 PI.addRange(R);
Owen Andersond382f8a2008-03-26 03:03:23 +00001042 }
Owen Anderson59df8782008-04-02 02:12:45 +00001043
Owen Andersonc7c00362008-03-29 01:58:47 +00001044 LI.RemoveMachineInstrFromMaps(PInstr);
1045 PInstr->eraseFromParent();
Owen Andersone46611e2008-03-24 04:11:27 +00001046 }
Owen Andersoncae8d8d2007-12-22 04:59:10 +00001047
Lang Hames233a60e2009-11-03 23:52:08 +00001048 LI.renumber();
Owen Anderson3947e4d2008-05-30 18:38:26 +00001049
Owen Andersonc7c00362008-03-29 01:58:47 +00001050 return true;
Owen Andersona4ad2e72007-11-06 04:49:43 +00001051}