blob: 5658a65abc879733123c66d0a1beca040e6f0dcd [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>();
Owen Andersoneb37ecc2008-03-10 07:22:36 +000075 AU.addRequired<LiveIntervals>();
76
77 // TODO: Actually make this true.
78 AU.addPreserved<LiveIntervals>();
Owen Anderson3947e4d2008-05-30 18:38:26 +000079 AU.addPreserved<RegisterCoalescer>();
Owen Anderson0bda0e82007-10-31 03:37:57 +000080 MachineFunctionPass::getAnalysisUsage(AU);
81 }
82
83 virtual void releaseMemory() {
84 preorder.clear();
85 maxpreorder.clear();
Owen Andersona4ad2e72007-11-06 04:49:43 +000086
Owen Andersonefbcebc2007-12-23 15:37:26 +000087 Waiting.clear();
Owen Andersonec1213f2008-01-09 22:40:54 +000088 Stacks.clear();
89 UsedByAnother.clear();
90 RenameSets.clear();
Owen Anderson0bda0e82007-10-31 03:37:57 +000091 }
92
93 private:
Owen Andersonec1213f2008-01-09 22:40:54 +000094
95 /// DomForestNode - Represents a node in the "dominator forest". This is
96 /// a forest in which the nodes represent registers and the edges
97 /// represent a dominance relation in the block defining those registers.
Owen Anderson83430bc2007-11-04 22:33:26 +000098 struct DomForestNode {
99 private:
Owen Andersonec1213f2008-01-09 22:40:54 +0000100 // Store references to our children
Owen Anderson83430bc2007-11-04 22:33:26 +0000101 std::vector<DomForestNode*> children;
Owen Andersonec1213f2008-01-09 22:40:54 +0000102 // The register we represent
Owen Andersonee49b532007-11-06 05:22:43 +0000103 unsigned reg;
Owen Anderson83430bc2007-11-04 22:33:26 +0000104
Owen Andersonec1213f2008-01-09 22:40:54 +0000105 // Add another node as our child
Owen Anderson83430bc2007-11-04 22:33:26 +0000106 void addChild(DomForestNode* DFN) { children.push_back(DFN); }
107
108 public:
109 typedef std::vector<DomForestNode*>::iterator iterator;
110
Owen Andersonec1213f2008-01-09 22:40:54 +0000111 // Create a DomForestNode by providing the register it represents, and
112 // the node to be its parent. The virtual root node has register 0
113 // and a null parent.
Owen Andersonee49b532007-11-06 05:22:43 +0000114 DomForestNode(unsigned r, DomForestNode* parent) : reg(r) {
Owen Anderson83430bc2007-11-04 22:33:26 +0000115 if (parent)
116 parent->addChild(this);
117 }
118
Owen Andersona4ad2e72007-11-06 04:49:43 +0000119 ~DomForestNode() {
120 for (iterator I = begin(), E = end(); I != E; ++I)
121 delete *I;
122 }
Owen Anderson83430bc2007-11-04 22:33:26 +0000123
Owen Andersonec1213f2008-01-09 22:40:54 +0000124 /// getReg - Return the regiser that this node represents
Owen Andersonee49b532007-11-06 05:22:43 +0000125 inline unsigned getReg() { return reg; }
Owen Andersona4ad2e72007-11-06 04:49:43 +0000126
Owen Andersonec1213f2008-01-09 22:40:54 +0000127 // Provide iterator access to our children
Owen Andersona4ad2e72007-11-06 04:49:43 +0000128 inline DomForestNode::iterator begin() { return children.begin(); }
129 inline DomForestNode::iterator end() { return children.end(); }
Owen Anderson83430bc2007-11-04 22:33:26 +0000130 };
131
Owen Anderson0bda0e82007-10-31 03:37:57 +0000132 void computeDFS(MachineFunction& MF);
Owen Anderson60a877d2007-11-07 05:17:15 +0000133 void processBlock(MachineBasicBlock* MBB);
Owen Anderson83430bc2007-11-04 22:33:26 +0000134
Owen Andersonc12417e2008-07-24 17:12:16 +0000135 std::vector<DomForestNode*> computeDomForest(
136 std::map<unsigned, MachineBasicBlock*>& instrs,
Owen Andersonddd060f2008-01-10 01:36:43 +0000137 MachineRegisterInfo& MRI);
Owen Andersond525f662007-12-11 20:12:11 +0000138 void processPHIUnion(MachineInstr* Inst,
Owen Andersonc12417e2008-07-24 17:12:16 +0000139 std::map<unsigned, MachineBasicBlock*>& PHIUnion,
Owen Anderson62d67dd2007-12-13 05:53:03 +0000140 std::vector<StrongPHIElimination::DomForestNode*>& DF,
141 std::vector<std::pair<unsigned, unsigned> >& locals);
Owen Andersonf1519e82007-12-24 22:12:23 +0000142 void ScheduleCopies(MachineBasicBlock* MBB, std::set<unsigned>& pushed);
Owen Anderson724acf12008-08-07 18:28:07 +0000143 void InsertCopies(MachineDomTreeNode* MBB,
Owen Andersone46611e2008-03-24 04:11:27 +0000144 SmallPtrSet<MachineBasicBlock*, 16>& v);
Owen Andersonfae86ed2008-09-23 04:37:10 +0000145 bool mergeLiveIntervals(unsigned primary, unsigned secondary);
Owen Anderson0bda0e82007-10-31 03:37:57 +0000146 };
Owen Anderson0bda0e82007-10-31 03:37:57 +0000147}
148
Dan Gohman844731a2008-05-13 00:00:25 +0000149char StrongPHIElimination::ID = 0;
150static RegisterPass<StrongPHIElimination>
151X("strong-phi-node-elimination",
152 "Eliminate PHI nodes for register allocation, intelligently");
153
Dan Gohman6ddba2b2008-05-13 02:05:11 +0000154const PassInfo *const llvm::StrongPHIEliminationID = &X;
Owen Anderson0bda0e82007-10-31 03:37:57 +0000155
156/// computeDFS - Computes the DFS-in and DFS-out numbers of the dominator tree
157/// of the given MachineFunction. These numbers are then used in other parts
158/// of the PHI elimination process.
159void StrongPHIElimination::computeDFS(MachineFunction& MF) {
160 SmallPtrSet<MachineDomTreeNode*, 8> frontier;
161 SmallPtrSet<MachineDomTreeNode*, 8> visited;
162
163 unsigned time = 0;
164
165 MachineDominatorTree& DT = getAnalysis<MachineDominatorTree>();
166
167 MachineDomTreeNode* node = DT.getRootNode();
168
169 std::vector<MachineDomTreeNode*> worklist;
170 worklist.push_back(node);
171
172 while (!worklist.empty()) {
173 MachineDomTreeNode* currNode = worklist.back();
174
175 if (!frontier.count(currNode)) {
176 frontier.insert(currNode);
177 ++time;
178 preorder.insert(std::make_pair(currNode->getBlock(), time));
179 }
180
181 bool inserted = false;
Owen Andersonbc91bd32008-03-31 01:39:20 +0000182 for (MachineDomTreeNode::iterator I = currNode->begin(), E = currNode->end();
Owen Anderson0bda0e82007-10-31 03:37:57 +0000183 I != E; ++I)
184 if (!frontier.count(*I) && !visited.count(*I)) {
185 worklist.push_back(*I);
186 inserted = true;
187 break;
188 }
189
190 if (!inserted) {
191 frontier.erase(currNode);
192 visited.insert(currNode);
193 maxpreorder.insert(std::make_pair(currNode->getBlock(), time));
194
195 worklist.pop_back();
196 }
197 }
Duncan Sands1bd32712007-10-31 08:49:24 +0000198}
Owen Anderson83430bc2007-11-04 22:33:26 +0000199
Dan Gohman844731a2008-05-13 00:00:25 +0000200namespace {
201
Owen Anderson8b96b9f2007-11-06 05:26:02 +0000202/// PreorderSorter - a helper class that is used to sort registers
203/// according to the preorder number of their defining blocks
Owen Anderson83430bc2007-11-04 22:33:26 +0000204class PreorderSorter {
205private:
206 DenseMap<MachineBasicBlock*, unsigned>& preorder;
Owen Andersonddd060f2008-01-10 01:36:43 +0000207 MachineRegisterInfo& MRI;
Owen Anderson83430bc2007-11-04 22:33:26 +0000208
209public:
Owen Andersonee49b532007-11-06 05:22:43 +0000210 PreorderSorter(DenseMap<MachineBasicBlock*, unsigned>& p,
Owen Andersonddd060f2008-01-10 01:36:43 +0000211 MachineRegisterInfo& M) : preorder(p), MRI(M) { }
Owen Anderson83430bc2007-11-04 22:33:26 +0000212
Owen Andersonee49b532007-11-06 05:22:43 +0000213 bool operator()(unsigned A, unsigned B) {
Owen Anderson83430bc2007-11-04 22:33:26 +0000214 if (A == B)
215 return false;
216
Owen Andersonddd060f2008-01-10 01:36:43 +0000217 MachineBasicBlock* ABlock = MRI.getVRegDef(A)->getParent();
218 MachineBasicBlock* BBlock = MRI.getVRegDef(B)->getParent();
Owen Andersonee49b532007-11-06 05:22:43 +0000219
220 if (preorder[ABlock] < preorder[BBlock])
Owen Anderson83430bc2007-11-04 22:33:26 +0000221 return true;
Owen Andersonee49b532007-11-06 05:22:43 +0000222 else if (preorder[ABlock] > preorder[BBlock])
Owen Anderson83430bc2007-11-04 22:33:26 +0000223 return false;
224
Owen Andersonee49b532007-11-06 05:22:43 +0000225 return false;
Owen Anderson83430bc2007-11-04 22:33:26 +0000226 }
227};
228
Dan Gohman844731a2008-05-13 00:00:25 +0000229}
230
Owen Anderson8b96b9f2007-11-06 05:26:02 +0000231/// computeDomForest - compute the subforest of the DomTree corresponding
232/// to the defining blocks of the registers in question
Owen Anderson83430bc2007-11-04 22:33:26 +0000233std::vector<StrongPHIElimination::DomForestNode*>
Owen Andersonc12417e2008-07-24 17:12:16 +0000234StrongPHIElimination::computeDomForest(
235 std::map<unsigned, MachineBasicBlock*>& regs,
Owen Andersonddd060f2008-01-10 01:36:43 +0000236 MachineRegisterInfo& MRI) {
Owen Andersonec1213f2008-01-09 22:40:54 +0000237 // Begin by creating a virtual root node, since the actual results
238 // may well be a forest. Assume this node has maximum DFS-out number.
Owen Anderson83430bc2007-11-04 22:33:26 +0000239 DomForestNode* VirtualRoot = new DomForestNode(0, 0);
240 maxpreorder.insert(std::make_pair((MachineBasicBlock*)0, ~0UL));
241
Owen Andersonec1213f2008-01-09 22:40:54 +0000242 // Populate a worklist with the registers
Owen Andersonee49b532007-11-06 05:22:43 +0000243 std::vector<unsigned> worklist;
244 worklist.reserve(regs.size());
Owen Andersonc12417e2008-07-24 17:12:16 +0000245 for (std::map<unsigned, MachineBasicBlock*>::iterator I = regs.begin(),
246 E = regs.end(); I != E; ++I)
Owen Anderson00316712008-03-12 03:13:29 +0000247 worklist.push_back(I->first);
Owen Andersonee49b532007-11-06 05:22:43 +0000248
Owen Andersonec1213f2008-01-09 22:40:54 +0000249 // Sort the registers by the DFS-in number of their defining block
Owen Andersonddd060f2008-01-10 01:36:43 +0000250 PreorderSorter PS(preorder, MRI);
Owen Anderson83430bc2007-11-04 22:33:26 +0000251 std::sort(worklist.begin(), worklist.end(), PS);
252
Owen Andersonec1213f2008-01-09 22:40:54 +0000253 // Create a "current parent" stack, and put the virtual root on top of it
Owen Anderson83430bc2007-11-04 22:33:26 +0000254 DomForestNode* CurrentParent = VirtualRoot;
255 std::vector<DomForestNode*> stack;
256 stack.push_back(VirtualRoot);
257
Owen Andersonec1213f2008-01-09 22:40:54 +0000258 // Iterate over all the registers in the previously computed order
Owen Andersonee49b532007-11-06 05:22:43 +0000259 for (std::vector<unsigned>::iterator I = worklist.begin(), E = worklist.end();
260 I != E; ++I) {
Owen Andersonddd060f2008-01-10 01:36:43 +0000261 unsigned pre = preorder[MRI.getVRegDef(*I)->getParent()];
Owen Andersoncb7d9492008-01-09 06:19:05 +0000262 MachineBasicBlock* parentBlock = CurrentParent->getReg() ?
Owen Andersonddd060f2008-01-10 01:36:43 +0000263 MRI.getVRegDef(CurrentParent->getReg())->getParent() :
Owen Andersoncb7d9492008-01-09 06:19:05 +0000264 0;
Owen Andersonee49b532007-11-06 05:22:43 +0000265
Owen Andersonec1213f2008-01-09 22:40:54 +0000266 // If the DFS-in number of the register is greater than the DFS-out number
267 // of the current parent, repeatedly pop the parent stack until it isn't.
Owen Andersonee49b532007-11-06 05:22:43 +0000268 while (pre > maxpreorder[parentBlock]) {
Owen Anderson83430bc2007-11-04 22:33:26 +0000269 stack.pop_back();
270 CurrentParent = stack.back();
Owen Andersonee49b532007-11-06 05:22:43 +0000271
Owen Anderson864e3a32008-01-09 10:41:39 +0000272 parentBlock = CurrentParent->getReg() ?
Owen Andersonddd060f2008-01-10 01:36:43 +0000273 MRI.getVRegDef(CurrentParent->getReg())->getParent() :
Owen Anderson864e3a32008-01-09 10:41:39 +0000274 0;
Owen Anderson83430bc2007-11-04 22:33:26 +0000275 }
276
Owen Andersonec1213f2008-01-09 22:40:54 +0000277 // Now that we've found the appropriate parent, create a DomForestNode for
278 // this register and attach it to the forest
Owen Anderson83430bc2007-11-04 22:33:26 +0000279 DomForestNode* child = new DomForestNode(*I, CurrentParent);
Owen Andersonec1213f2008-01-09 22:40:54 +0000280
281 // Push this new node on the "current parent" stack
Owen Anderson83430bc2007-11-04 22:33:26 +0000282 stack.push_back(child);
283 CurrentParent = child;
284 }
285
Owen Andersonec1213f2008-01-09 22:40:54 +0000286 // Return a vector containing the children of the virtual root node
Owen Anderson83430bc2007-11-04 22:33:26 +0000287 std::vector<DomForestNode*> ret;
288 ret.insert(ret.end(), VirtualRoot->begin(), VirtualRoot->end());
289 return ret;
290}
Owen Andersona4ad2e72007-11-06 04:49:43 +0000291
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000292/// isLiveIn - helper method that determines, from a regno, if a register
Owen Anderson60a877d2007-11-07 05:17:15 +0000293/// is live into a block
Owen Andersonddd060f2008-01-10 01:36:43 +0000294static bool isLiveIn(unsigned r, MachineBasicBlock* MBB,
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000295 LiveIntervals& LI) {
296 LiveInterval& I = LI.getOrCreateInterval(r);
Lang Hamescc3b0652009-10-03 04:21:37 +0000297 LiveIndex idx = LI.getMBBStartIdx(MBB);
Owen Andersond55fccd2008-10-12 20:39:30 +0000298 return I.liveAt(idx);
Owen Anderson60a877d2007-11-07 05:17:15 +0000299}
300
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000301/// isLiveOut - help method that determines, from a regno, if a register is
Owen Anderson60a877d2007-11-07 05:17:15 +0000302/// live out of a block.
Owen Andersonddd060f2008-01-10 01:36:43 +0000303static bool isLiveOut(unsigned r, MachineBasicBlock* MBB,
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000304 LiveIntervals& LI) {
305 for (MachineBasicBlock::succ_iterator PI = MBB->succ_begin(),
Owen Anderson724acf12008-08-07 18:28:07 +0000306 E = MBB->succ_end(); PI != E; ++PI)
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000307 if (isLiveIn(r, *PI, LI))
308 return true;
Owen Anderson60a877d2007-11-07 05:17:15 +0000309
310 return false;
311}
312
Owen Anderson87a702b2007-12-16 05:44:27 +0000313/// interferes - checks for local interferences by scanning a block. The only
314/// trick parameter is 'mode' which tells it the relationship of the two
315/// registers. 0 - defined in the same block, 1 - first properly dominates
316/// second, 2 - second properly dominates first
Owen Andersonb199cbe2008-01-10 00:33:11 +0000317static bool interferes(unsigned a, unsigned b, MachineBasicBlock* scan,
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000318 LiveIntervals& LV, unsigned mode) {
Owen Anderson87a702b2007-12-16 05:44:27 +0000319 MachineInstr* def = 0;
320 MachineInstr* kill = 0;
321
Owen Andersonddd060f2008-01-10 01:36:43 +0000322 // The code is still in SSA form at this point, so there is only one
323 // definition per VReg. Thus we can safely use MRI->getVRegDef().
324 const MachineRegisterInfo* MRI = &scan->getParent()->getRegInfo();
Owen Andersonb199cbe2008-01-10 00:33:11 +0000325
Owen Anderson87a702b2007-12-16 05:44:27 +0000326 bool interference = false;
327
328 // Wallk the block, checking for interferences
329 for (MachineBasicBlock::iterator MBI = scan->begin(), MBE = scan->end();
330 MBI != MBE; ++MBI) {
331 MachineInstr* curr = MBI;
332
333 // Same defining block...
334 if (mode == 0) {
Owen Andersonddd060f2008-01-10 01:36:43 +0000335 if (curr == MRI->getVRegDef(a)) {
336 // If we find our first definition, save it
Owen Anderson87a702b2007-12-16 05:44:27 +0000337 if (!def) {
338 def = curr;
Owen Andersonddd060f2008-01-10 01:36:43 +0000339 // If there's already an unkilled definition, then
Owen Anderson87a702b2007-12-16 05:44:27 +0000340 // this is an interference
341 } else if (!kill) {
342 interference = true;
343 break;
Owen Andersonddd060f2008-01-10 01:36:43 +0000344 // If there's a definition followed by a KillInst, then
Owen Anderson87a702b2007-12-16 05:44:27 +0000345 // they can't interfere
346 } else {
347 interference = false;
348 break;
349 }
350 // Symmetric with the above
Owen Andersonddd060f2008-01-10 01:36:43 +0000351 } else if (curr == MRI->getVRegDef(b)) {
Owen Anderson87a702b2007-12-16 05:44:27 +0000352 if (!def) {
353 def = curr;
354 } else if (!kill) {
355 interference = true;
356 break;
357 } else {
358 interference = false;
359 break;
360 }
Owen Andersonddd060f2008-01-10 01:36:43 +0000361 // Store KillInsts if they match up with the definition
Evan Cheng6130f662008-03-05 00:59:57 +0000362 } else if (curr->killsRegister(a)) {
Owen Andersonddd060f2008-01-10 01:36:43 +0000363 if (def == MRI->getVRegDef(a)) {
Owen Anderson87a702b2007-12-16 05:44:27 +0000364 kill = curr;
Evan Cheng6130f662008-03-05 00:59:57 +0000365 } else if (curr->killsRegister(b)) {
Owen Andersonddd060f2008-01-10 01:36:43 +0000366 if (def == MRI->getVRegDef(b)) {
Owen Anderson87a702b2007-12-16 05:44:27 +0000367 kill = curr;
368 }
369 }
370 }
371 // First properly dominates second...
372 } else if (mode == 1) {
Owen Andersonddd060f2008-01-10 01:36:43 +0000373 if (curr == MRI->getVRegDef(b)) {
374 // Definition of second without kill of first is an interference
Owen Anderson87a702b2007-12-16 05:44:27 +0000375 if (!kill) {
376 interference = true;
377 break;
Owen Andersonddd060f2008-01-10 01:36:43 +0000378 // Definition after a kill is a non-interference
Owen Anderson87a702b2007-12-16 05:44:27 +0000379 } else {
380 interference = false;
381 break;
382 }
383 // Save KillInsts of First
Evan Cheng6130f662008-03-05 00:59:57 +0000384 } else if (curr->killsRegister(a)) {
Owen Anderson87a702b2007-12-16 05:44:27 +0000385 kill = curr;
386 }
387 // Symmetric with the above
388 } else if (mode == 2) {
Owen Andersonddd060f2008-01-10 01:36:43 +0000389 if (curr == MRI->getVRegDef(a)) {
Owen Anderson87a702b2007-12-16 05:44:27 +0000390 if (!kill) {
391 interference = true;
392 break;
393 } else {
394 interference = false;
395 break;
396 }
Evan Cheng6130f662008-03-05 00:59:57 +0000397 } else if (curr->killsRegister(b)) {
Owen Anderson87a702b2007-12-16 05:44:27 +0000398 kill = curr;
399 }
400 }
401 }
402
403 return interference;
404}
405
Owen Andersondc4d6552008-01-10 00:47:01 +0000406/// processBlock - Determine how to break up PHIs in the current block. Each
407/// PHI is broken up by some combination of renaming its operands and inserting
408/// copies. This method is responsible for determining which operands receive
409/// which treatment.
Owen Anderson60a877d2007-11-07 05:17:15 +0000410void StrongPHIElimination::processBlock(MachineBasicBlock* MBB) {
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000411 LiveIntervals& LI = getAnalysis<LiveIntervals>();
Owen Andersonddd060f2008-01-10 01:36:43 +0000412 MachineRegisterInfo& MRI = MBB->getParent()->getRegInfo();
Owen Anderson60a877d2007-11-07 05:17:15 +0000413
414 // Holds names that have been added to a set in any PHI within this block
415 // before the current one.
416 std::set<unsigned> ProcessedNames;
417
Owen Andersondc4d6552008-01-10 00:47:01 +0000418 // Iterate over all the PHI nodes in this block
Owen Anderson60a877d2007-11-07 05:17:15 +0000419 MachineBasicBlock::iterator P = MBB->begin();
Owen Anderson78216bc2008-04-01 18:05:08 +0000420 while (P != MBB->end() && P->getOpcode() == TargetInstrInfo::PHI) {
Owen Andersonafc6de02007-12-10 08:07:09 +0000421 unsigned DestReg = P->getOperand(0).getReg();
Owen Anderson9860b712008-08-06 20:29:20 +0000422
Owen Andersond382f8a2008-03-26 03:03:23 +0000423 // Don't both doing PHI elimination for dead PHI's.
424 if (P->registerDefIsDead(DestReg)) {
425 ++P;
426 continue;
427 }
428
Owen Andersondfd07ea2008-03-12 04:22:57 +0000429 LiveInterval& PI = LI.getOrCreateInterval(DestReg);
Lang Hamescc3b0652009-10-03 04:21:37 +0000430 LiveIndex pIdx = LI.getDefIndex(LI.getInstructionIndex(P));
Owen Andersondfd07ea2008-03-12 04:22:57 +0000431 VNInfo* PVN = PI.getLiveRangeContaining(pIdx)->valno;
432 PhiValueNumber.insert(std::make_pair(DestReg, PVN->id));
433
Owen Andersondc4d6552008-01-10 00:47:01 +0000434 // PHIUnion is the set of incoming registers to the PHI node that
435 // are going to be renames rather than having copies inserted. This set
436 // is refinded over the course of this function. UnionedBlocks is the set
437 // of corresponding MBBs.
Owen Andersonc12417e2008-07-24 17:12:16 +0000438 std::map<unsigned, MachineBasicBlock*> PHIUnion;
Owen Andersone46611e2008-03-24 04:11:27 +0000439 SmallPtrSet<MachineBasicBlock*, 8> UnionedBlocks;
Owen Anderson60a877d2007-11-07 05:17:15 +0000440
Owen Andersondc4d6552008-01-10 00:47:01 +0000441 // Iterate over the operands of the PHI node
Owen Anderson60a877d2007-11-07 05:17:15 +0000442 for (int i = P->getNumOperands() - 1; i >= 2; i-=2) {
443 unsigned SrcReg = P->getOperand(i-1).getReg();
Owen Anderson9860b712008-08-06 20:29:20 +0000444
445 // Don't need to try to coalesce a register with itself.
446 if (SrcReg == DestReg) {
447 ProcessedNames.insert(SrcReg);
448 continue;
449 }
Owen Anderson082d3512008-09-26 18:50:46 +0000450
451 // We don't need to insert copies for implicit_defs.
452 MachineInstr* DefMI = MRI.getVRegDef(SrcReg);
453 if (DefMI->getOpcode() == TargetInstrInfo::IMPLICIT_DEF)
454 ProcessedNames.insert(SrcReg);
Owen Anderson60a877d2007-11-07 05:17:15 +0000455
Owen Andersondc4d6552008-01-10 00:47:01 +0000456 // Check for trivial interferences via liveness information, allowing us
457 // to avoid extra work later. Any registers that interfere cannot both
458 // be in the renaming set, so choose one and add copies for it instead.
459 // The conditions are:
460 // 1) if the operand is live into the PHI node's block OR
461 // 2) if the PHI node is live out of the operand's defining block OR
462 // 3) if the operand is itself a PHI node and the original PHI is
463 // live into the operand's defining block OR
464 // 4) if the operand is already being renamed for another PHI node
465 // in this block OR
466 // 5) if any two operands are defined in the same block, insert copies
467 // for one of them
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000468 if (isLiveIn(SrcReg, P->getParent(), LI) ||
Owen Andersonddd060f2008-01-10 01:36:43 +0000469 isLiveOut(P->getOperand(0).getReg(),
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000470 MRI.getVRegDef(SrcReg)->getParent(), LI) ||
Owen Andersonddd060f2008-01-10 01:36:43 +0000471 ( MRI.getVRegDef(SrcReg)->getOpcode() == TargetInstrInfo::PHI &&
472 isLiveIn(P->getOperand(0).getReg(),
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000473 MRI.getVRegDef(SrcReg)->getParent(), LI) ) ||
Owen Andersonafc6de02007-12-10 08:07:09 +0000474 ProcessedNames.count(SrcReg) ||
Owen Andersonddd060f2008-01-10 01:36:43 +0000475 UnionedBlocks.count(MRI.getVRegDef(SrcReg)->getParent())) {
Owen Andersonafc6de02007-12-10 08:07:09 +0000476
Owen Andersondc4d6552008-01-10 00:47:01 +0000477 // Add a copy for the selected register
Chris Lattner8aa797a2007-12-30 23:10:15 +0000478 MachineBasicBlock* From = P->getOperand(i).getMBB();
Owen Andersonefbcebc2007-12-23 15:37:26 +0000479 Waiting[From].insert(std::make_pair(SrcReg, DestReg));
480 UsedByAnother.insert(SrcReg);
Owen Anderson60a877d2007-11-07 05:17:15 +0000481 } else {
Owen Andersondc4d6552008-01-10 00:47:01 +0000482 // Otherwise, add it to the renaming set
Owen Andersonc12417e2008-07-24 17:12:16 +0000483 PHIUnion.insert(std::make_pair(SrcReg,P->getOperand(i).getMBB()));
Owen Andersonddd060f2008-01-10 01:36:43 +0000484 UnionedBlocks.insert(MRI.getVRegDef(SrcReg)->getParent());
Owen Anderson60a877d2007-11-07 05:17:15 +0000485 }
Owen Anderson60a877d2007-11-07 05:17:15 +0000486 }
487
Owen Andersondc4d6552008-01-10 00:47:01 +0000488 // Compute the dominator forest for the renaming set. This is a forest
489 // where the nodes are the registers and the edges represent dominance
490 // relations between the defining blocks of the registers
Owen Anderson42f9e962007-11-13 20:13:24 +0000491 std::vector<StrongPHIElimination::DomForestNode*> DF =
Owen Andersonddd060f2008-01-10 01:36:43 +0000492 computeDomForest(PHIUnion, MRI);
Owen Anderson42f9e962007-11-13 20:13:24 +0000493
Owen Andersondc4d6552008-01-10 00:47:01 +0000494 // Walk DomForest to resolve interferences at an inter-block level. This
495 // will remove registers from the renaming set (and insert copies for them)
496 // if interferences are found.
Owen Anderson62d67dd2007-12-13 05:53:03 +0000497 std::vector<std::pair<unsigned, unsigned> > localInterferences;
498 processPHIUnion(P, PHIUnion, DF, localInterferences);
499
Owen Anderson52b17332008-04-02 03:00:13 +0000500 // If one of the inputs is defined in the same block as the current PHI
501 // then we need to check for a local interference between that input and
502 // the PHI.
Owen Andersonc12417e2008-07-24 17:12:16 +0000503 for (std::map<unsigned, MachineBasicBlock*>::iterator I = PHIUnion.begin(),
Owen Anderson52b17332008-04-02 03:00:13 +0000504 E = PHIUnion.end(); I != E; ++I)
505 if (MRI.getVRegDef(I->first)->getParent() == P->getParent())
506 localInterferences.push_back(std::make_pair(I->first,
507 P->getOperand(0).getReg()));
508
Owen Andersondc4d6552008-01-10 00:47:01 +0000509 // The dominator forest walk may have returned some register pairs whose
Owen Anderson52b17332008-04-02 03:00:13 +0000510 // interference cannot be determined from dominator analysis. We now
Owen Andersondc4d6552008-01-10 00:47:01 +0000511 // examine these pairs for local interferences.
Owen Anderson87a702b2007-12-16 05:44:27 +0000512 for (std::vector<std::pair<unsigned, unsigned> >::iterator I =
513 localInterferences.begin(), E = localInterferences.end(); I != E; ++I) {
514 std::pair<unsigned, unsigned> p = *I;
515
Owen Anderson87a702b2007-12-16 05:44:27 +0000516 MachineDominatorTree& MDT = getAnalysis<MachineDominatorTree>();
517
518 // Determine the block we need to scan and the relationship between
519 // the two registers
520 MachineBasicBlock* scan = 0;
521 unsigned mode = 0;
Owen Andersonddd060f2008-01-10 01:36:43 +0000522 if (MRI.getVRegDef(p.first)->getParent() ==
523 MRI.getVRegDef(p.second)->getParent()) {
524 scan = MRI.getVRegDef(p.first)->getParent();
Owen Anderson87a702b2007-12-16 05:44:27 +0000525 mode = 0; // Same block
Owen Andersonddd060f2008-01-10 01:36:43 +0000526 } else if (MDT.dominates(MRI.getVRegDef(p.first)->getParent(),
527 MRI.getVRegDef(p.second)->getParent())) {
528 scan = MRI.getVRegDef(p.second)->getParent();
Owen Anderson87a702b2007-12-16 05:44:27 +0000529 mode = 1; // First dominates second
530 } else {
Owen Andersonddd060f2008-01-10 01:36:43 +0000531 scan = MRI.getVRegDef(p.first)->getParent();
Owen Anderson87a702b2007-12-16 05:44:27 +0000532 mode = 2; // Second dominates first
533 }
534
535 // If there's an interference, we need to insert copies
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000536 if (interferes(p.first, p.second, scan, LI, mode)) {
Owen Anderson87a702b2007-12-16 05:44:27 +0000537 // Insert copies for First
538 for (int i = P->getNumOperands() - 1; i >= 2; i-=2) {
539 if (P->getOperand(i-1).getReg() == p.first) {
540 unsigned SrcReg = p.first;
541 MachineBasicBlock* From = P->getOperand(i).getMBB();
542
Owen Andersonefbcebc2007-12-23 15:37:26 +0000543 Waiting[From].insert(std::make_pair(SrcReg,
544 P->getOperand(0).getReg()));
545 UsedByAnother.insert(SrcReg);
546
Owen Anderson87a702b2007-12-16 05:44:27 +0000547 PHIUnion.erase(SrcReg);
548 }
549 }
550 }
551 }
Owen Anderson42f9e962007-11-13 20:13:24 +0000552
Owen Anderson52b17332008-04-02 03:00:13 +0000553 // Add the renaming set for this PHI node to our overall renaming information
Owen Andersone04cec02008-09-22 21:58:58 +0000554 for (std::map<unsigned, MachineBasicBlock*>::iterator QI = PHIUnion.begin(),
555 QE = PHIUnion.end(); QI != QE; ++QI) {
Chris Lattnerbbbfa992009-08-23 06:35:02 +0000556 DEBUG(errs() << "Adding Renaming: " << QI->first << " -> "
557 << P->getOperand(0).getReg() << "\n");
Owen Andersone04cec02008-09-22 21:58:58 +0000558 }
559
Owen Anderson0c5714b2008-01-08 21:54:52 +0000560 RenameSets.insert(std::make_pair(P->getOperand(0).getReg(), PHIUnion));
Owen Andersoncae8d8d2007-12-22 04:59:10 +0000561
Owen Andersondc4d6552008-01-10 00:47:01 +0000562 // Remember which registers are already renamed, so that we don't try to
563 // rename them for another PHI node in this block
Owen Andersonc12417e2008-07-24 17:12:16 +0000564 for (std::map<unsigned, MachineBasicBlock*>::iterator I = PHIUnion.begin(),
Owen Anderson00316712008-03-12 03:13:29 +0000565 E = PHIUnion.end(); I != E; ++I)
566 ProcessedNames.insert(I->first);
Owen Andersondc4d6552008-01-10 00:47:01 +0000567
Owen Anderson60a877d2007-11-07 05:17:15 +0000568 ++P;
569 }
Owen Andersonee49b532007-11-06 05:22:43 +0000570}
571
Gabor Greif2cf36e02008-03-06 10:51:21 +0000572/// processPHIUnion - Take a set of candidate registers to be coalesced when
Owen Anderson965b4672007-12-16 04:07:23 +0000573/// decomposing the PHI instruction. Use the DominanceForest to remove the ones
574/// that are known to interfere, and flag others that need to be checked for
575/// local interferences.
Owen Andersond525f662007-12-11 20:12:11 +0000576void StrongPHIElimination::processPHIUnion(MachineInstr* Inst,
Owen Andersonc12417e2008-07-24 17:12:16 +0000577 std::map<unsigned, MachineBasicBlock*>& PHIUnion,
Owen Anderson62d67dd2007-12-13 05:53:03 +0000578 std::vector<StrongPHIElimination::DomForestNode*>& DF,
579 std::vector<std::pair<unsigned, unsigned> >& locals) {
Owen Andersond525f662007-12-11 20:12:11 +0000580
581 std::vector<DomForestNode*> worklist(DF.begin(), DF.end());
582 SmallPtrSet<DomForestNode*, 4> visited;
583
Owen Andersonddd060f2008-01-10 01:36:43 +0000584 // Code is still in SSA form, so we can use MRI::getVRegDef()
585 MachineRegisterInfo& MRI = Inst->getParent()->getParent()->getRegInfo();
586
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000587 LiveIntervals& LI = getAnalysis<LiveIntervals>();
Owen Andersond525f662007-12-11 20:12:11 +0000588 unsigned DestReg = Inst->getOperand(0).getReg();
589
Owen Anderson965b4672007-12-16 04:07:23 +0000590 // DF walk on the DomForest
Owen Andersond525f662007-12-11 20:12:11 +0000591 while (!worklist.empty()) {
592 DomForestNode* DFNode = worklist.back();
593
Owen Andersond525f662007-12-11 20:12:11 +0000594 visited.insert(DFNode);
595
596 bool inserted = false;
Owen Andersond525f662007-12-11 20:12:11 +0000597 for (DomForestNode::iterator CI = DFNode->begin(), CE = DFNode->end();
598 CI != CE; ++CI) {
599 DomForestNode* child = *CI;
Owen Anderson3b489522008-01-21 22:01:01 +0000600
601 // If the current node is live-out of the defining block of one of its
Owen Andersona6b19262008-01-21 22:03:00 +0000602 // children, insert a copy for it. NOTE: The paper actually calls for
603 // a more elaborate heuristic for determining whether to insert copies
604 // for the child or the parent. In the interest of simplicity, we're
605 // just always choosing the parent.
Owen Andersonddd060f2008-01-10 01:36:43 +0000606 if (isLiveOut(DFNode->getReg(),
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000607 MRI.getVRegDef(child->getReg())->getParent(), LI)) {
Owen Andersond525f662007-12-11 20:12:11 +0000608 // Insert copies for parent
609 for (int i = Inst->getNumOperands() - 1; i >= 2; i-=2) {
610 if (Inst->getOperand(i-1).getReg() == DFNode->getReg()) {
Owen Andersoned2ffa22007-12-12 01:25:08 +0000611 unsigned SrcReg = DFNode->getReg();
Owen Andersond525f662007-12-11 20:12:11 +0000612 MachineBasicBlock* From = Inst->getOperand(i).getMBB();
613
Owen Andersonefbcebc2007-12-23 15:37:26 +0000614 Waiting[From].insert(std::make_pair(SrcReg, DestReg));
615 UsedByAnother.insert(SrcReg);
616
Owen Andersoned2ffa22007-12-12 01:25:08 +0000617 PHIUnion.erase(SrcReg);
Owen Andersond525f662007-12-11 20:12:11 +0000618 }
619 }
Owen Anderson3b489522008-01-21 22:01:01 +0000620
621 // If a node is live-in to the defining block of one of its children, but
622 // not live-out, then we need to scan that block for local interferences.
Owen Andersonddd060f2008-01-10 01:36:43 +0000623 } else if (isLiveIn(DFNode->getReg(),
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000624 MRI.getVRegDef(child->getReg())->getParent(), LI) ||
Owen Andersonddd060f2008-01-10 01:36:43 +0000625 MRI.getVRegDef(DFNode->getReg())->getParent() ==
626 MRI.getVRegDef(child->getReg())->getParent()) {
Owen Anderson62d67dd2007-12-13 05:53:03 +0000627 // Add (p, c) to possible local interferences
628 locals.push_back(std::make_pair(DFNode->getReg(), child->getReg()));
Owen Andersond525f662007-12-11 20:12:11 +0000629 }
Owen Anderson965b4672007-12-16 04:07:23 +0000630
Owen Anderson4ba08ec2007-12-13 05:43:37 +0000631 if (!visited.count(child)) {
632 worklist.push_back(child);
633 inserted = true;
Owen Andersond525f662007-12-11 20:12:11 +0000634 }
635 }
636
637 if (!inserted) worklist.pop_back();
638 }
639}
640
Owen Andersonefbcebc2007-12-23 15:37:26 +0000641/// ScheduleCopies - Insert copies into predecessor blocks, scheduling
642/// them properly so as to avoid the 'lost copy' and the 'virtual swap'
643/// problems.
644///
645/// Based on "Practical Improvements to the Construction and Destruction
646/// of Static Single Assignment Form" by Briggs, et al.
Owen Andersonf1519e82007-12-24 22:12:23 +0000647void StrongPHIElimination::ScheduleCopies(MachineBasicBlock* MBB,
648 std::set<unsigned>& pushed) {
Owen Anderson38b42502008-06-04 17:55:58 +0000649 // FIXME: This function needs to update LiveIntervals
Owen Anderson8481ee72008-10-02 19:40:33 +0000650 std::multimap<unsigned, unsigned>& copy_set= Waiting[MBB];
Owen Andersonefbcebc2007-12-23 15:37:26 +0000651
Owen Anderson8481ee72008-10-02 19:40:33 +0000652 std::multimap<unsigned, unsigned> worklist;
Owen Andersonefbcebc2007-12-23 15:37:26 +0000653 std::map<unsigned, unsigned> map;
654
655 // Setup worklist of initial copies
Argyrios Kyrtzidis0304b872008-10-04 08:11:49 +0000656 for (std::multimap<unsigned, unsigned>::iterator I = copy_set.begin(),
Owen Andersonefbcebc2007-12-23 15:37:26 +0000657 E = copy_set.end(); I != E; ) {
658 map.insert(std::make_pair(I->first, I->first));
659 map.insert(std::make_pair(I->second, I->second));
660
Owen Andersonc7c00362008-03-29 01:58:47 +0000661 if (!UsedByAnother.count(I->second)) {
Owen Andersonefbcebc2007-12-23 15:37:26 +0000662 worklist.insert(*I);
663
664 // Avoid iterator invalidation
Owen Anderson8481ee72008-10-02 19:40:33 +0000665 std::multimap<unsigned, unsigned>::iterator OI = I;
Owen Andersonefbcebc2007-12-23 15:37:26 +0000666 ++I;
Owen Anderson8481ee72008-10-02 19:40:33 +0000667 copy_set.erase(OI);
Owen Andersonefbcebc2007-12-23 15:37:26 +0000668 } else {
669 ++I;
670 }
671 }
672
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000673 LiveIntervals& LI = getAnalysis<LiveIntervals>();
Owen Anderson0d893b42008-01-08 05:16:15 +0000674 MachineFunction* MF = MBB->getParent();
Owen Andersonddd060f2008-01-10 01:36:43 +0000675 MachineRegisterInfo& MRI = MF->getRegInfo();
Owen Anderson0d893b42008-01-08 05:16:15 +0000676 const TargetInstrInfo *TII = MF->getTarget().getInstrInfo();
Owen Andersonefbcebc2007-12-23 15:37:26 +0000677
Owen Anderson38b42502008-06-04 17:55:58 +0000678 SmallVector<std::pair<unsigned, MachineInstr*>, 4> InsertedPHIDests;
679
Owen Andersonefbcebc2007-12-23 15:37:26 +0000680 // Iterate over the worklist, inserting copies
681 while (!worklist.empty() || !copy_set.empty()) {
682 while (!worklist.empty()) {
Owen Anderson8481ee72008-10-02 19:40:33 +0000683 std::multimap<unsigned, unsigned>::iterator WI = worklist.begin();
684 std::pair<unsigned, unsigned> curr = *WI;
685 worklist.erase(WI);
Owen Andersonefbcebc2007-12-23 15:37:26 +0000686
Owen Anderson0d893b42008-01-08 05:16:15 +0000687 const TargetRegisterClass *RC = MF->getRegInfo().getRegClass(curr.first);
688
Owen Andersoneb37ecc2008-03-10 07:22:36 +0000689 if (isLiveOut(curr.second, MBB, LI)) {
Owen Anderson0d893b42008-01-08 05:16:15 +0000690 // Create a temporary
691 unsigned t = MF->getRegInfo().createVirtualRegister(RC);
692
693 // Insert copy from curr.second to a temporary at
694 // the Phi defining curr.second
Owen Andersonddd060f2008-01-10 01:36:43 +0000695 MachineBasicBlock::iterator PI = MRI.getVRegDef(curr.second);
696 TII->copyRegToReg(*PI->getParent(), PI, t,
Owen Anderson0d893b42008-01-08 05:16:15 +0000697 curr.second, RC, RC);
698
Chris Lattnerbbbfa992009-08-23 06:35:02 +0000699 DEBUG(errs() << "Inserted copy from " << curr.second << " to " << t
700 << "\n");
Owen Anderson8481ee72008-10-02 19:40:33 +0000701
Owen Andersonefbcebc2007-12-23 15:37:26 +0000702 // Push temporary on Stacks
Owen Anderson0d893b42008-01-08 05:16:15 +0000703 Stacks[curr.second].push_back(t);
704
705 // Insert curr.second in pushed
706 pushed.insert(curr.second);
Owen Anderson724acf12008-08-07 18:28:07 +0000707
708 // Create a live interval for this temporary
709 InsertedPHIDests.push_back(std::make_pair(t, --PI));
Owen Andersonefbcebc2007-12-23 15:37:26 +0000710 }
711
712 // Insert copy from map[curr.first] to curr.second
Owen Anderson9c2efa82008-01-10 00:01:41 +0000713 TII->copyRegToReg(*MBB, MBB->getFirstTerminator(), curr.second,
Owen Anderson0d893b42008-01-08 05:16:15 +0000714 map[curr.first], RC, RC);
Owen Andersonefbcebc2007-12-23 15:37:26 +0000715 map[curr.first] = curr.second;
Chris Lattnerbbbfa992009-08-23 06:35:02 +0000716 DEBUG(errs() << "Inserted copy from " << curr.first << " to "
717 << curr.second << "\n");
Owen Andersonefbcebc2007-12-23 15:37:26 +0000718
Owen Anderson38b42502008-06-04 17:55:58 +0000719 // Push this copy onto InsertedPHICopies so we can
720 // update LiveIntervals with it.
721 MachineBasicBlock::iterator MI = MBB->getFirstTerminator();
722 InsertedPHIDests.push_back(std::make_pair(curr.second, --MI));
723
Owen Andersonefbcebc2007-12-23 15:37:26 +0000724 // If curr.first is a destination in copy_set...
Owen Anderson8481ee72008-10-02 19:40:33 +0000725 for (std::multimap<unsigned, unsigned>::iterator I = copy_set.begin(),
Owen Andersonefbcebc2007-12-23 15:37:26 +0000726 E = copy_set.end(); I != E; )
727 if (curr.first == I->second) {
728 std::pair<unsigned, unsigned> temp = *I;
Owen Anderson8481ee72008-10-02 19:40:33 +0000729 worklist.insert(temp);
Owen Andersonefbcebc2007-12-23 15:37:26 +0000730
731 // Avoid iterator invalidation
Owen Anderson8481ee72008-10-02 19:40:33 +0000732 std::multimap<unsigned, unsigned>::iterator OI = I;
Owen Andersonefbcebc2007-12-23 15:37:26 +0000733 ++I;
Owen Anderson8481ee72008-10-02 19:40:33 +0000734 copy_set.erase(OI);
Owen Andersonefbcebc2007-12-23 15:37:26 +0000735
736 break;
737 } else {
738 ++I;
739 }
740 }
741
742 if (!copy_set.empty()) {
Owen Anderson8481ee72008-10-02 19:40:33 +0000743 std::multimap<unsigned, unsigned>::iterator CI = copy_set.begin();
744 std::pair<unsigned, unsigned> curr = *CI;
Owen Andersonefbcebc2007-12-23 15:37:26 +0000745 worklist.insert(curr);
Owen Anderson8481ee72008-10-02 19:40:33 +0000746 copy_set.erase(CI);
Owen Anderson27d66812008-08-13 17:25:42 +0000747
748 LiveInterval& I = LI.getInterval(curr.second);
749 MachineBasicBlock::iterator term = MBB->getFirstTerminator();
Lang Hames6cc91e32009-10-03 04:31:31 +0000750 LiveIndex endIdx = LiveIndex();
Owen Anderson27d66812008-08-13 17:25:42 +0000751 if (term != MBB->end())
752 endIdx = LI.getInstructionIndex(term);
753 else
754 endIdx = LI.getMBBEndIdx(MBB);
755
756 if (I.liveAt(endIdx)) {
757 const TargetRegisterClass *RC =
758 MF->getRegInfo().getRegClass(curr.first);
759
760 // Insert a copy from dest to a new temporary t at the end of b
761 unsigned t = MF->getRegInfo().createVirtualRegister(RC);
762 TII->copyRegToReg(*MBB, MBB->getFirstTerminator(), t,
763 curr.second, RC, RC);
764 map[curr.second] = t;
765
766 MachineBasicBlock::iterator TI = MBB->getFirstTerminator();
767 InsertedPHIDests.push_back(std::make_pair(t, --TI));
768 }
Owen Andersonefbcebc2007-12-23 15:37:26 +0000769 }
770 }
Owen Anderson38b42502008-06-04 17:55:58 +0000771
772 // Renumber the instructions so that we can perform the index computations
773 // needed to create new live intervals.
774 LI.computeNumbering();
775
776 // For copies that we inserted at the ends of predecessors, we construct
777 // live intervals. This is pretty easy, since we know that the destination
778 // register cannot have be in live at that point previously. We just have
779 // to make sure that, for registers that serve as inputs to more than one
780 // PHI, we don't create multiple overlapping live intervals.
781 std::set<unsigned> RegHandled;
782 for (SmallVector<std::pair<unsigned, MachineInstr*>, 4>::iterator I =
Owen Anderson27d66812008-08-13 17:25:42 +0000783 InsertedPHIDests.begin(), E = InsertedPHIDests.end(); I != E; ++I) {
Owen Andersone04cec02008-09-22 21:58:58 +0000784 if (RegHandled.insert(I->first).second) {
785 LiveInterval& Int = LI.getOrCreateInterval(I->first);
Lang Hamescc3b0652009-10-03 04:21:37 +0000786 LiveIndex instrIdx = LI.getInstructionIndex(I->second);
Lang Hames96479942009-09-09 20:14:17 +0000787 if (Int.liveAt(LI.getDefIndex(instrIdx)))
788 Int.removeRange(LI.getDefIndex(instrIdx),
Lang Hames86511252009-09-04 20:41:11 +0000789 LI.getNextSlot(LI.getMBBEndIdx(I->second->getParent())),
Owen Andersone04cec02008-09-22 21:58:58 +0000790 true);
791
792 LiveRange R = LI.addLiveRangeToEndOfBlock(I->first, I->second);
Lang Hames52c1afc2009-08-10 23:43:28 +0000793 R.valno->setCopy(I->second);
Lang Hames96479942009-09-09 20:14:17 +0000794 R.valno->def = LI.getDefIndex(LI.getInstructionIndex(I->second));
Owen Andersone04cec02008-09-22 21:58:58 +0000795 }
Owen Anderson27d66812008-08-13 17:25:42 +0000796 }
Owen Andersonefbcebc2007-12-23 15:37:26 +0000797}
798
Owen Andersonf1519e82007-12-24 22:12:23 +0000799/// InsertCopies - insert copies into MBB and all of its successors
Owen Anderson724acf12008-08-07 18:28:07 +0000800void StrongPHIElimination::InsertCopies(MachineDomTreeNode* MDTN,
Owen Andersone46611e2008-03-24 04:11:27 +0000801 SmallPtrSet<MachineBasicBlock*, 16>& visited) {
Owen Anderson724acf12008-08-07 18:28:07 +0000802 MachineBasicBlock* MBB = MDTN->getBlock();
Owen Anderson719fef62008-01-09 10:32:30 +0000803 visited.insert(MBB);
804
Owen Andersonf1519e82007-12-24 22:12:23 +0000805 std::set<unsigned> pushed;
806
Owen Anderson724acf12008-08-07 18:28:07 +0000807 LiveIntervals& LI = getAnalysis<LiveIntervals>();
Owen Andersonf1519e82007-12-24 22:12:23 +0000808 // Rewrite register uses from Stacks
809 for (MachineBasicBlock::iterator I = MBB->begin(), E = MBB->end();
Owen Anderson724acf12008-08-07 18:28:07 +0000810 I != E; ++I) {
811 if (I->getOpcode() == TargetInstrInfo::PHI)
812 continue;
813
Owen Andersonf1519e82007-12-24 22:12:23 +0000814 for (unsigned i = 0; i < I->getNumOperands(); ++i)
Dan Gohmand735b802008-10-03 15:45:36 +0000815 if (I->getOperand(i).isReg() &&
Owen Andersonf1519e82007-12-24 22:12:23 +0000816 Stacks[I->getOperand(i).getReg()].size()) {
Owen Anderson724acf12008-08-07 18:28:07 +0000817 // Remove the live range for the old vreg.
818 LiveInterval& OldInt = LI.getInterval(I->getOperand(i).getReg());
819 LiveInterval::iterator OldLR = OldInt.FindLiveRangeContaining(
Lang Hames96479942009-09-09 20:14:17 +0000820 LI.getUseIndex(LI.getInstructionIndex(I)));
Owen Anderson724acf12008-08-07 18:28:07 +0000821 if (OldLR != OldInt.end())
822 OldInt.removeRange(*OldLR, true);
823
824 // Change the register
Owen Andersonf1519e82007-12-24 22:12:23 +0000825 I->getOperand(i).setReg(Stacks[I->getOperand(i).getReg()].back());
Owen Anderson724acf12008-08-07 18:28:07 +0000826
827 // Add a live range for the new vreg
828 LiveInterval& Int = LI.getInterval(I->getOperand(i).getReg());
829 VNInfo* FirstVN = *Int.vni_begin();
Lang Hames857c4e02009-06-17 21:01:20 +0000830 FirstVN->setHasPHIKill(false);
Owen Anderson724acf12008-08-07 18:28:07 +0000831 if (I->getOperand(i).isKill())
Lang Hames86511252009-09-04 20:41:11 +0000832 FirstVN->addKill(
Lang Hames96479942009-09-09 20:14:17 +0000833 LI.getUseIndex(LI.getInstructionIndex(I)));
Owen Anderson724acf12008-08-07 18:28:07 +0000834
835 LiveRange LR (LI.getMBBStartIdx(I->getParent()),
Lang Hames86511252009-09-04 20:41:11 +0000836 LI.getNextSlot(LI.getUseIndex(LI.getInstructionIndex(I))),
Owen Anderson724acf12008-08-07 18:28:07 +0000837 FirstVN);
838
839 Int.addRange(LR);
Owen Andersonf1519e82007-12-24 22:12:23 +0000840 }
Owen Anderson724acf12008-08-07 18:28:07 +0000841 }
Owen Andersonf1519e82007-12-24 22:12:23 +0000842
843 // Schedule the copies for this block
844 ScheduleCopies(MBB, pushed);
845
Owen Anderson724acf12008-08-07 18:28:07 +0000846 // Recur down the dominator tree.
847 for (MachineDomTreeNode::iterator I = MDTN->begin(),
848 E = MDTN->end(); I != E; ++I)
849 if (!visited.count((*I)->getBlock()))
Owen Anderson719fef62008-01-09 10:32:30 +0000850 InsertCopies(*I, visited);
Owen Andersonf1519e82007-12-24 22:12:23 +0000851
852 // As we exit this block, pop the names we pushed while processing it
853 for (std::set<unsigned>::iterator I = pushed.begin(),
854 E = pushed.end(); I != E; ++I)
855 Stacks[*I].pop_back();
856}
857
Owen Andersonfae86ed2008-09-23 04:37:10 +0000858bool StrongPHIElimination::mergeLiveIntervals(unsigned primary,
Owen Anderson27d66812008-08-13 17:25:42 +0000859 unsigned secondary) {
Owen Anderson755ebab2008-03-17 06:08:26 +0000860
861 LiveIntervals& LI = getAnalysis<LiveIntervals>();
862 LiveInterval& LHS = LI.getOrCreateInterval(primary);
863 LiveInterval& RHS = LI.getOrCreateInterval(secondary);
864
Owen Andersona9efb262008-06-05 17:22:53 +0000865 LI.computeNumbering();
Owen Andersone04cec02008-09-22 21:58:58 +0000866
Owen Andersonb9fb8d12008-07-30 00:21:16 +0000867 DenseMap<VNInfo*, VNInfo*> VNMap;
Owen Andersone04cec02008-09-22 21:58:58 +0000868 for (LiveInterval::iterator I = RHS.begin(), E = RHS.end(); I != E; ++I) {
869 LiveRange R = *I;
870
Lang Hamescc3b0652009-10-03 04:21:37 +0000871 LiveIndex Start = R.start;
872 LiveIndex End = R.end;
Owen Andersonfae86ed2008-09-23 04:37:10 +0000873 if (LHS.getLiveRangeContaining(Start))
874 return false;
Owen Andersone04cec02008-09-22 21:58:58 +0000875
Owen Andersonfae86ed2008-09-23 04:37:10 +0000876 if (LHS.getLiveRangeContaining(End))
877 return false;
Owen Andersone04cec02008-09-22 21:58:58 +0000878
879 LiveInterval::iterator RI = std::upper_bound(LHS.begin(), LHS.end(), R);
880 if (RI != LHS.end() && RI->start < End)
Owen Andersonfae86ed2008-09-23 04:37:10 +0000881 return false;
882 }
883
884 for (LiveInterval::iterator I = RHS.begin(), E = RHS.end(); I != E; ++I) {
885 LiveRange R = *I;
886 VNInfo* OldVN = R.valno;
887 VNInfo*& NewVN = VNMap[OldVN];
888 if (!NewVN) {
Lang Hames857c4e02009-06-17 21:01:20 +0000889 NewVN = LHS.createValueCopy(OldVN, LI.getVNInfoAllocator());
Owen Andersone04cec02008-09-22 21:58:58 +0000890 }
Owen Andersonfae86ed2008-09-23 04:37:10 +0000891
892 LiveRange LR (R.start, R.end, NewVN);
893 LHS.addRange(LR);
Owen Andersonb9fb8d12008-07-30 00:21:16 +0000894 }
Owen Anderson55c64352008-07-25 21:08:41 +0000895
Owen Andersone04cec02008-09-22 21:58:58 +0000896 LI.removeInterval(RHS.reg);
Owen Andersonfae86ed2008-09-23 04:37:10 +0000897
898 return true;
Owen Anderson00316712008-03-12 03:13:29 +0000899}
900
Owen Andersona4ad2e72007-11-06 04:49:43 +0000901bool StrongPHIElimination::runOnMachineFunction(MachineFunction &Fn) {
Owen Andersonc7c00362008-03-29 01:58:47 +0000902 LiveIntervals& LI = getAnalysis<LiveIntervals>();
903
Owen Andersonefbcebc2007-12-23 15:37:26 +0000904 // Compute DFS numbers of each block
Owen Andersona4ad2e72007-11-06 04:49:43 +0000905 computeDFS(Fn);
906
Owen Andersonefbcebc2007-12-23 15:37:26 +0000907 // Determine which phi node operands need copies
Owen Anderson60a877d2007-11-07 05:17:15 +0000908 for (MachineFunction::iterator I = Fn.begin(), E = Fn.end(); I != E; ++I)
909 if (!I->empty() &&
910 I->begin()->getOpcode() == TargetInstrInfo::PHI)
911 processBlock(I);
Owen Andersona4ad2e72007-11-06 04:49:43 +0000912
Owen Anderson883771f2008-08-06 22:08:58 +0000913 // Break interferences where two different phis want to coalesce
914 // in the same register.
915 std::set<unsigned> seen;
916 typedef std::map<unsigned, std::map<unsigned, MachineBasicBlock*> >
917 RenameSetType;
918 for (RenameSetType::iterator I = RenameSets.begin(), E = RenameSets.end();
919 I != E; ++I) {
920 for (std::map<unsigned, MachineBasicBlock*>::iterator
921 OI = I->second.begin(), OE = I->second.end(); OI != OE; ) {
922 if (!seen.count(OI->first)) {
923 seen.insert(OI->first);
924 ++OI;
925 } else {
926 Waiting[OI->second].insert(std::make_pair(OI->first, I->first));
927 unsigned reg = OI->first;
928 ++OI;
929 I->second.erase(reg);
Chris Lattnerbbbfa992009-08-23 06:35:02 +0000930 DEBUG(errs() << "Removing Renaming: " << reg << " -> " << I->first
931 << "\n");
Owen Anderson883771f2008-08-06 22:08:58 +0000932 }
933 }
934 }
935
Owen Andersonefbcebc2007-12-23 15:37:26 +0000936 // Insert copies
Owen Andersona9efb262008-06-05 17:22:53 +0000937 // FIXME: This process should probably preserve LiveIntervals
Owen Andersone46611e2008-03-24 04:11:27 +0000938 SmallPtrSet<MachineBasicBlock*, 16> visited;
Owen Anderson724acf12008-08-07 18:28:07 +0000939 MachineDominatorTree& MDT = getAnalysis<MachineDominatorTree>();
940 InsertCopies(MDT.getRootNode(), visited);
Owen Andersonefbcebc2007-12-23 15:37:26 +0000941
Owen Anderson0c5714b2008-01-08 21:54:52 +0000942 // Perform renaming
Owen Anderson0c5714b2008-01-08 21:54:52 +0000943 for (RenameSetType::iterator I = RenameSets.begin(), E = RenameSets.end();
944 I != E; ++I)
Owen Anderson883771f2008-08-06 22:08:58 +0000945 while (I->second.size()) {
946 std::map<unsigned, MachineBasicBlock*>::iterator SI = I->second.begin();
947
Chris Lattnerbbbfa992009-08-23 06:35:02 +0000948 DEBUG(errs() << "Renaming: " << SI->first << " -> " << I->first << "\n");
Owen Andersone04cec02008-09-22 21:58:58 +0000949
Owen Anderson883771f2008-08-06 22:08:58 +0000950 if (SI->first != I->first) {
Owen Andersonfae86ed2008-09-23 04:37:10 +0000951 if (mergeLiveIntervals(I->first, SI->first)) {
952 Fn.getRegInfo().replaceRegWith(SI->first, I->first);
Owen Anderson883771f2008-08-06 22:08:58 +0000953
Owen Andersonfae86ed2008-09-23 04:37:10 +0000954 if (RenameSets.count(SI->first)) {
955 I->second.insert(RenameSets[SI->first].begin(),
956 RenameSets[SI->first].end());
957 RenameSets.erase(SI->first);
958 }
959 } else {
960 // Insert a last-minute copy if a conflict was detected.
961 const TargetInstrInfo *TII = Fn.getTarget().getInstrInfo();
962 const TargetRegisterClass *RC = Fn.getRegInfo().getRegClass(I->first);
963 TII->copyRegToReg(*SI->second, SI->second->getFirstTerminator(),
964 I->first, SI->first, RC, RC);
965
966 LI.computeNumbering();
967
968 LiveInterval& Int = LI.getOrCreateInterval(I->first);
Lang Hamescc3b0652009-10-03 04:21:37 +0000969 LiveIndex instrIdx =
Owen Andersonfae86ed2008-09-23 04:37:10 +0000970 LI.getInstructionIndex(--SI->second->getFirstTerminator());
Lang Hames96479942009-09-09 20:14:17 +0000971 if (Int.liveAt(LI.getDefIndex(instrIdx)))
972 Int.removeRange(LI.getDefIndex(instrIdx),
Lang Hames86511252009-09-04 20:41:11 +0000973 LI.getNextSlot(LI.getMBBEndIdx(SI->second)), true);
Owen Andersonfae86ed2008-09-23 04:37:10 +0000974
975 LiveRange R = LI.addLiveRangeToEndOfBlock(I->first,
976 --SI->second->getFirstTerminator());
Lang Hames52c1afc2009-08-10 23:43:28 +0000977 R.valno->setCopy(--SI->second->getFirstTerminator());
Lang Hames96479942009-09-09 20:14:17 +0000978 R.valno->def = LI.getDefIndex(instrIdx);
Owen Andersonfae86ed2008-09-23 04:37:10 +0000979
Chris Lattnerbbbfa992009-08-23 06:35:02 +0000980 DEBUG(errs() << "Renaming failed: " << SI->first << " -> "
981 << I->first << "\n");
Owen Anderson883771f2008-08-06 22:08:58 +0000982 }
Owen Anderson883771f2008-08-06 22:08:58 +0000983 }
984
Owen Anderson05472ee2008-09-30 23:58:47 +0000985 LiveInterval& Int = LI.getOrCreateInterval(I->first);
986 const LiveRange* LR =
987 Int.getLiveRangeContaining(LI.getMBBEndIdx(SI->second));
Lang Hames857c4e02009-06-17 21:01:20 +0000988 LR->valno->setHasPHIKill(true);
Owen Anderson05472ee2008-09-30 23:58:47 +0000989
Owen Anderson883771f2008-08-06 22:08:58 +0000990 I->second.erase(SI->first);
Owen Anderson00316712008-03-12 03:13:29 +0000991 }
Owen Anderson0c5714b2008-01-08 21:54:52 +0000992
Owen Anderson0c5714b2008-01-08 21:54:52 +0000993 // Remove PHIs
Owen Anderson97ca75e2008-01-22 23:58:54 +0000994 std::vector<MachineInstr*> phis;
995 for (MachineFunction::iterator I = Fn.begin(), E = Fn.end(); I != E; ++I) {
Owen Anderson0c5714b2008-01-08 21:54:52 +0000996 for (MachineBasicBlock::iterator BI = I->begin(), BE = I->end();
997 BI != BE; ++BI)
998 if (BI->getOpcode() == TargetInstrInfo::PHI)
Owen Anderson97ca75e2008-01-22 23:58:54 +0000999 phis.push_back(BI);
1000 }
1001
1002 for (std::vector<MachineInstr*>::iterator I = phis.begin(), E = phis.end();
Owen Andersonc7c00362008-03-29 01:58:47 +00001003 I != E; ) {
1004 MachineInstr* PInstr = *(I++);
1005
Owen Anderson71ac0be2008-08-05 21:18:51 +00001006 // If this is a dead PHI node, then remove it from LiveIntervals.
1007 unsigned DestReg = PInstr->getOperand(0).getReg();
1008 LiveInterval& PI = LI.getInterval(DestReg);
1009 if (PInstr->registerDefIsDead(DestReg)) {
1010 if (PI.containsOneValue()) {
1011 LI.removeInterval(DestReg);
1012 } else {
Lang Hamescc3b0652009-10-03 04:21:37 +00001013 LiveIndex idx = LI.getDefIndex(LI.getInstructionIndex(PInstr));
Owen Anderson71ac0be2008-08-05 21:18:51 +00001014 PI.removeRange(*PI.getLiveRangeContaining(idx), true);
1015 }
1016 } else {
Owen Andersone7b82052008-08-05 20:51:26 +00001017 // Trim live intervals of input registers. They are no longer live into
Owen Anderson9b491202008-08-06 18:36:17 +00001018 // this block if they died after the PHI. If they lived after it, don't
1019 // trim them because they might have other legitimate uses.
Owen Andersone7b82052008-08-05 20:51:26 +00001020 for (unsigned i = 1; i < PInstr->getNumOperands(); i += 2) {
1021 unsigned reg = PInstr->getOperand(i).getReg();
Owen Anderson883771f2008-08-06 22:08:58 +00001022
Owen Andersone7b82052008-08-05 20:51:26 +00001023 MachineBasicBlock* MBB = PInstr->getOperand(i+1).getMBB();
1024 LiveInterval& InputI = LI.getInterval(reg);
1025 if (MBB != PInstr->getParent() &&
Owen Anderson9b491202008-08-06 18:36:17 +00001026 InputI.liveAt(LI.getMBBStartIdx(PInstr->getParent())) &&
Lang Hames86511252009-09-04 20:41:11 +00001027 InputI.expiredAt(LI.getNextIndex(LI.getInstructionIndex(PInstr))))
Owen Andersone7b82052008-08-05 20:51:26 +00001028 InputI.removeRange(LI.getMBBStartIdx(PInstr->getParent()),
1029 LI.getInstructionIndex(PInstr),
1030 true);
Owen Andersond382f8a2008-03-26 03:03:23 +00001031 }
Owen Andersone7b82052008-08-05 20:51:26 +00001032
Owen Anderson71ac0be2008-08-05 21:18:51 +00001033 // If the PHI is not dead, then the valno defined by the PHI
1034 // now has an unknown def.
Lang Hamescc3b0652009-10-03 04:21:37 +00001035 LiveIndex idx = LI.getDefIndex(LI.getInstructionIndex(PInstr));
Owen Anderson71ac0be2008-08-05 21:18:51 +00001036 const LiveRange* PLR = PI.getLiveRangeContaining(idx);
Lang Hames857c4e02009-06-17 21:01:20 +00001037 PLR->valno->setIsPHIDef(true);
Owen Anderson71ac0be2008-08-05 21:18:51 +00001038 LiveRange R (LI.getMBBStartIdx(PInstr->getParent()),
1039 PLR->start, PLR->valno);
1040 PI.addRange(R);
Owen Andersond382f8a2008-03-26 03:03:23 +00001041 }
Owen Anderson59df8782008-04-02 02:12:45 +00001042
Owen Andersonc7c00362008-03-29 01:58:47 +00001043 LI.RemoveMachineInstrFromMaps(PInstr);
1044 PInstr->eraseFromParent();
Owen Andersone46611e2008-03-24 04:11:27 +00001045 }
Owen Andersoncae8d8d2007-12-22 04:59:10 +00001046
Owen Anderson3947e4d2008-05-30 18:38:26 +00001047 LI.computeNumbering();
1048
Owen Andersonc7c00362008-03-29 01:58:47 +00001049 return true;
Owen Andersona4ad2e72007-11-06 04:49:43 +00001050}