blob: 246552127865a657e992689f3466eb515315bbf2 [file] [log] [blame]
Evan Chenga8e29892007-01-19 07:51:42 +00001//===-- ARMConstantIslandPass.cpp - ARM constant islands --------*- C++ -*-===//
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.
Evan Chenga8e29892007-01-19 07:51:42 +00007//
8//===----------------------------------------------------------------------===//
9//
10// This file contains a pass that splits the constant pool up into 'islands'
11// which are scattered through-out the function. This is required due to the
12// limited pc-relative displacements that ARM has.
13//
14//===----------------------------------------------------------------------===//
15
16#define DEBUG_TYPE "arm-cp-islands"
17#include "ARM.h"
Evan Chengd3d9d662009-07-23 18:27:47 +000018#include "ARMAddressingModes.h"
Evan Chengaf5cbcb2007-01-25 03:12:46 +000019#include "ARMMachineFunctionInfo.h"
Evan Chenga8e29892007-01-19 07:51:42 +000020#include "ARMInstrInfo.h"
21#include "llvm/CodeGen/MachineConstantPool.h"
22#include "llvm/CodeGen/MachineFunctionPass.h"
23#include "llvm/CodeGen/MachineInstrBuilder.h"
Evan Chenga8e29892007-01-19 07:51:42 +000024#include "llvm/Target/TargetData.h"
25#include "llvm/Target/TargetMachine.h"
26#include "llvm/Support/Compiler.h"
27#include "llvm/Support/Debug.h"
Torok Edwinc25e7582009-07-11 20:10:48 +000028#include "llvm/Support/ErrorHandling.h"
Evan Chengc99ef082007-02-09 20:54:44 +000029#include "llvm/ADT/SmallVector.h"
Evan Chenga8e29892007-01-19 07:51:42 +000030#include "llvm/ADT/STLExtras.h"
31#include "llvm/ADT/Statistic.h"
Evan Chenga8e29892007-01-19 07:51:42 +000032using namespace llvm;
33
Evan Chengc99ef082007-02-09 20:54:44 +000034STATISTIC(NumCPEs, "Number of constpool entries");
Evan Chengd1b2c1e2007-01-30 01:18:38 +000035STATISTIC(NumSplit, "Number of uncond branches inserted");
36STATISTIC(NumCBrFixed, "Number of cond branches fixed");
37STATISTIC(NumUBrFixed, "Number of uncond branches fixed");
Evan Chenga8e29892007-01-19 07:51:42 +000038
39namespace {
Dale Johannesen88e37ae2007-02-23 05:02:36 +000040 /// ARMConstantIslands - Due to limited PC-relative displacements, ARM
Evan Chenga8e29892007-01-19 07:51:42 +000041 /// requires constant pool entries to be scattered among the instructions
42 /// inside a function. To do this, it completely ignores the normal LLVM
Dale Johannesen88e37ae2007-02-23 05:02:36 +000043 /// constant pool; instead, it places constants wherever it feels like with
Evan Chenga8e29892007-01-19 07:51:42 +000044 /// special instructions.
45 ///
46 /// The terminology used in this pass includes:
47 /// Islands - Clumps of constants placed in the function.
48 /// Water - Potential places where an island could be formed.
49 /// CPE - A constant pool entry that has been placed somewhere, which
50 /// tracks a list of users.
51 class VISIBILITY_HIDDEN ARMConstantIslands : public MachineFunctionPass {
Evan Chenga8e29892007-01-19 07:51:42 +000052 /// BBSizes - The size of each MachineBasicBlock in bytes of code, indexed
Dale Johannesen8593e412007-04-29 19:19:30 +000053 /// by MBB Number. The two-byte pads required for Thumb alignment are
54 /// counted as part of the following block (i.e., the offset and size for
55 /// a padded block will both be ==2 mod 4).
Evan Chenge03cff62007-02-09 23:59:14 +000056 std::vector<unsigned> BBSizes;
Bob Wilson84945262009-05-12 17:09:30 +000057
Dale Johannesen99c49a42007-02-25 00:47:03 +000058 /// BBOffsets - the offset of each MBB in bytes, starting from 0.
Dale Johannesen8593e412007-04-29 19:19:30 +000059 /// The two-byte pads required for Thumb alignment are counted as part of
60 /// the following block.
Dale Johannesen99c49a42007-02-25 00:47:03 +000061 std::vector<unsigned> BBOffsets;
62
Evan Chenga8e29892007-01-19 07:51:42 +000063 /// WaterList - A sorted list of basic blocks where islands could be placed
64 /// (i.e. blocks that don't fall through to the following block, due
65 /// to a return, unreachable, or unconditional branch).
Evan Chenge03cff62007-02-09 23:59:14 +000066 std::vector<MachineBasicBlock*> WaterList;
Evan Chengc99ef082007-02-09 20:54:44 +000067
Evan Chenga8e29892007-01-19 07:51:42 +000068 /// CPUser - One user of a constant pool, keeping the machine instruction
69 /// pointer, the constant pool being referenced, and the max displacement
70 /// allowed from the instruction to the CP.
71 struct CPUser {
72 MachineInstr *MI;
73 MachineInstr *CPEMI;
74 unsigned MaxDisp;
Evan Cheng5d8f1ca2009-07-21 23:56:01 +000075 bool NegOk;
Evan Chengd3d9d662009-07-23 18:27:47 +000076 bool IsSoImm;
77 CPUser(MachineInstr *mi, MachineInstr *cpemi, unsigned maxdisp,
78 bool neg, bool soimm)
79 : MI(mi), CPEMI(cpemi), MaxDisp(maxdisp), NegOk(neg), IsSoImm(soimm) {}
Evan Chenga8e29892007-01-19 07:51:42 +000080 };
Bob Wilson84945262009-05-12 17:09:30 +000081
Evan Chenga8e29892007-01-19 07:51:42 +000082 /// CPUsers - Keep track of all of the machine instructions that use various
83 /// constant pools and their max displacement.
Evan Chenge03cff62007-02-09 23:59:14 +000084 std::vector<CPUser> CPUsers;
Bob Wilson84945262009-05-12 17:09:30 +000085
Evan Chengc99ef082007-02-09 20:54:44 +000086 /// CPEntry - One per constant pool entry, keeping the machine instruction
87 /// pointer, the constpool index, and the number of CPUser's which
88 /// reference this entry.
89 struct CPEntry {
90 MachineInstr *CPEMI;
91 unsigned CPI;
92 unsigned RefCount;
93 CPEntry(MachineInstr *cpemi, unsigned cpi, unsigned rc = 0)
94 : CPEMI(cpemi), CPI(cpi), RefCount(rc) {}
95 };
96
97 /// CPEntries - Keep track of all of the constant pool entry machine
Dale Johannesen88e37ae2007-02-23 05:02:36 +000098 /// instructions. For each original constpool index (i.e. those that
99 /// existed upon entry to this pass), it keeps a vector of entries.
100 /// Original elements are cloned as we go along; the clones are
101 /// put in the vector of the original element, but have distinct CPIs.
Evan Chengc99ef082007-02-09 20:54:44 +0000102 std::vector<std::vector<CPEntry> > CPEntries;
Bob Wilson84945262009-05-12 17:09:30 +0000103
Evan Chengaf5cbcb2007-01-25 03:12:46 +0000104 /// ImmBranch - One per immediate branch, keeping the machine instruction
105 /// pointer, conditional or unconditional, the max displacement,
106 /// and (if isCond is true) the corresponding unconditional branch
107 /// opcode.
108 struct ImmBranch {
109 MachineInstr *MI;
Evan Chengc2854142007-01-25 23:18:59 +0000110 unsigned MaxDisp : 31;
111 bool isCond : 1;
Evan Chengaf5cbcb2007-01-25 03:12:46 +0000112 int UncondBr;
Evan Chengc2854142007-01-25 23:18:59 +0000113 ImmBranch(MachineInstr *mi, unsigned maxdisp, bool cond, int ubr)
114 : MI(mi), MaxDisp(maxdisp), isCond(cond), UncondBr(ubr) {}
Evan Chengaf5cbcb2007-01-25 03:12:46 +0000115 };
116
Evan Cheng2706f972007-05-16 05:14:06 +0000117 /// ImmBranches - Keep track of all the immediate branch instructions.
Evan Chengaf5cbcb2007-01-25 03:12:46 +0000118 ///
Evan Chenge03cff62007-02-09 23:59:14 +0000119 std::vector<ImmBranch> ImmBranches;
Evan Chengaf5cbcb2007-01-25 03:12:46 +0000120
Evan Chengd1b2c1e2007-01-30 01:18:38 +0000121 /// PushPopMIs - Keep track of all the Thumb push / pop instructions.
122 ///
Evan Chengc99ef082007-02-09 20:54:44 +0000123 SmallVector<MachineInstr*, 4> PushPopMIs;
Evan Chengd1b2c1e2007-01-30 01:18:38 +0000124
125 /// HasFarJump - True if any far jump instruction has been emitted during
126 /// the branch fix up pass.
127 bool HasFarJump;
128
Evan Chenga8e29892007-01-19 07:51:42 +0000129 const TargetInstrInfo *TII;
Dale Johannesen8593e412007-04-29 19:19:30 +0000130 ARMFunctionInfo *AFI;
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000131 bool isThumb;
Evan Chengd3d9d662009-07-23 18:27:47 +0000132 bool isThumb1;
David Goodwin5e47a9a2009-06-30 18:04:13 +0000133 bool isThumb2;
Evan Chenga8e29892007-01-19 07:51:42 +0000134 public:
Devang Patel19974732007-05-03 01:11:54 +0000135 static char ID;
Dan Gohmanae73dc12008-09-04 17:05:41 +0000136 ARMConstantIslands() : MachineFunctionPass(&ID) {}
Devang Patel794fd752007-05-01 21:15:47 +0000137
Evan Chenga8e29892007-01-19 07:51:42 +0000138 virtual bool runOnMachineFunction(MachineFunction &Fn);
139
140 virtual const char *getPassName() const {
Evan Chengaf5cbcb2007-01-25 03:12:46 +0000141 return "ARM constant island placement and branch shortening pass";
Evan Chenga8e29892007-01-19 07:51:42 +0000142 }
Bob Wilson84945262009-05-12 17:09:30 +0000143
Evan Chenga8e29892007-01-19 07:51:42 +0000144 private:
145 void DoInitialPlacement(MachineFunction &Fn,
Evan Chenge03cff62007-02-09 23:59:14 +0000146 std::vector<MachineInstr*> &CPEMIs);
Evan Chengc99ef082007-02-09 20:54:44 +0000147 CPEntry *findConstPoolEntry(unsigned CPI, const MachineInstr *CPEMI);
Evan Chenga8e29892007-01-19 07:51:42 +0000148 void InitialFunctionScan(MachineFunction &Fn,
Evan Chenge03cff62007-02-09 23:59:14 +0000149 const std::vector<MachineInstr*> &CPEMIs);
Evan Cheng0c615842007-01-31 02:22:22 +0000150 MachineBasicBlock *SplitBlockBeforeInstr(MachineInstr *MI);
Evan Chenga8e29892007-01-19 07:51:42 +0000151 void UpdateForInsertedWaterBlock(MachineBasicBlock *NewBB);
Dale Johannesen99c49a42007-02-25 00:47:03 +0000152 void AdjustBBOffsetsAfter(MachineBasicBlock *BB, int delta);
Evan Chenged884f32007-04-03 23:39:48 +0000153 bool DecrementOldEntry(unsigned CPI, MachineInstr* CPEMI);
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000154 int LookForExistingCPEntry(CPUser& U, unsigned UserOffset);
Bob Wilson84945262009-05-12 17:09:30 +0000155 bool LookForWater(CPUser&U, unsigned UserOffset,
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000156 MachineBasicBlock** NewMBB);
Bob Wilson84945262009-05-12 17:09:30 +0000157 MachineBasicBlock* AcceptWater(MachineBasicBlock *WaterBB,
Dale Johannesen8593e412007-04-29 19:19:30 +0000158 std::vector<MachineBasicBlock*>::iterator IP);
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000159 void CreateNewWater(unsigned CPUserIndex, unsigned UserOffset,
160 MachineBasicBlock** NewMBB);
Dale Johannesenf1b214d2007-02-28 18:41:23 +0000161 bool HandleConstantPoolUser(MachineFunction &Fn, unsigned CPUserIndex);
Evan Chenged884f32007-04-03 23:39:48 +0000162 void RemoveDeadCPEMI(MachineInstr *CPEMI);
163 bool RemoveUnusedCPEntries();
Bob Wilson84945262009-05-12 17:09:30 +0000164 bool CPEIsInRange(MachineInstr *MI, unsigned UserOffset,
Evan Cheng5d8f1ca2009-07-21 23:56:01 +0000165 MachineInstr *CPEMI, unsigned Disp, bool NegOk,
166 bool DoDump = false);
Dale Johannesen99c49a42007-02-25 00:47:03 +0000167 bool WaterIsInRange(unsigned UserOffset, MachineBasicBlock *Water,
Dale Johannesen5d9c4b62007-07-11 18:32:38 +0000168 CPUser &U);
Dale Johannesen99c49a42007-02-25 00:47:03 +0000169 bool OffsetIsInRange(unsigned UserOffset, unsigned TrialOffset,
Evan Chengd3d9d662009-07-23 18:27:47 +0000170 unsigned Disp, bool NegativeOK, bool IsSoImm = false);
Evan Chengc0dbec72007-01-31 19:57:44 +0000171 bool BBIsInRange(MachineInstr *MI, MachineBasicBlock *BB, unsigned Disp);
Evan Chengd1b2c1e2007-01-30 01:18:38 +0000172 bool FixUpImmediateBr(MachineFunction &Fn, ImmBranch &Br);
173 bool FixUpConditionalBr(MachineFunction &Fn, ImmBranch &Br);
174 bool FixUpUnconditionalBr(MachineFunction &Fn, ImmBranch &Br);
175 bool UndoLRSpillRestore();
Evan Chenga8e29892007-01-19 07:51:42 +0000176
Evan Chenga8e29892007-01-19 07:51:42 +0000177 unsigned GetOffsetOf(MachineInstr *MI) const;
Dale Johannesen8593e412007-04-29 19:19:30 +0000178 void dumpBBs();
179 void verify(MachineFunction &Fn);
Evan Chenga8e29892007-01-19 07:51:42 +0000180 };
Devang Patel19974732007-05-03 01:11:54 +0000181 char ARMConstantIslands::ID = 0;
Evan Chenga8e29892007-01-19 07:51:42 +0000182}
183
Dale Johannesen8593e412007-04-29 19:19:30 +0000184/// verify - check BBOffsets, BBSizes, alignment of islands
185void ARMConstantIslands::verify(MachineFunction &Fn) {
186 assert(BBOffsets.size() == BBSizes.size());
187 for (unsigned i = 1, e = BBOffsets.size(); i != e; ++i)
188 assert(BBOffsets[i-1]+BBSizes[i-1] == BBOffsets[i]);
Evan Chengd3d9d662009-07-23 18:27:47 +0000189 if (!isThumb)
190 return;
191#ifndef NDEBUG
192 for (MachineFunction::iterator MBBI = Fn.begin(), E = Fn.end();
193 MBBI != E; ++MBBI) {
194 MachineBasicBlock *MBB = MBBI;
195 if (!MBB->empty() &&
196 MBB->begin()->getOpcode() == ARM::CONSTPOOL_ENTRY) {
197 unsigned MBBId = MBB->getNumber();
198 assert((BBOffsets[MBBId]%4 == 0 && BBSizes[MBBId]%4 == 0) ||
199 (BBOffsets[MBBId]%4 != 0 && BBSizes[MBBId]%4 != 0));
Dale Johannesen8593e412007-04-29 19:19:30 +0000200 }
201 }
Evan Chengd3d9d662009-07-23 18:27:47 +0000202#endif
Dale Johannesen8593e412007-04-29 19:19:30 +0000203}
204
205/// print block size and offset information - debugging
206void ARMConstantIslands::dumpBBs() {
207 for (unsigned J = 0, E = BBOffsets.size(); J !=E; ++J) {
Bob Wilson84945262009-05-12 17:09:30 +0000208 DOUT << "block " << J << " offset " << BBOffsets[J] <<
Dale Johannesen5d9c4b62007-07-11 18:32:38 +0000209 " size " << BBSizes[J] << "\n";
Dale Johannesen8593e412007-04-29 19:19:30 +0000210 }
211}
212
Evan Chengaf5cbcb2007-01-25 03:12:46 +0000213/// createARMConstantIslandPass - returns an instance of the constpool
214/// island pass.
Evan Chenga8e29892007-01-19 07:51:42 +0000215FunctionPass *llvm::createARMConstantIslandPass() {
216 return new ARMConstantIslands();
217}
218
219bool ARMConstantIslands::runOnMachineFunction(MachineFunction &Fn) {
Evan Chenga8e29892007-01-19 07:51:42 +0000220 MachineConstantPool &MCP = *Fn.getConstantPool();
Bob Wilson84945262009-05-12 17:09:30 +0000221
Evan Chenga8e29892007-01-19 07:51:42 +0000222 TII = Fn.getTarget().getInstrInfo();
Dale Johannesen8593e412007-04-29 19:19:30 +0000223 AFI = Fn.getInfo<ARMFunctionInfo>();
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000224 isThumb = AFI->isThumbFunction();
Evan Chengd3d9d662009-07-23 18:27:47 +0000225 isThumb1 = AFI->isThumb1OnlyFunction();
David Goodwin5e47a9a2009-06-30 18:04:13 +0000226 isThumb2 = AFI->isThumb2Function();
Evan Chengd1b2c1e2007-01-30 01:18:38 +0000227
228 HasFarJump = false;
229
Evan Chenga8e29892007-01-19 07:51:42 +0000230 // Renumber all of the machine basic blocks in the function, guaranteeing that
231 // the numbers agree with the position of the block in the function.
232 Fn.RenumberBlocks();
233
Evan Chengd3d9d662009-07-23 18:27:47 +0000234 // Thumb1 functions containing constant pools get 2-byte alignment.
235 // This is so we can keep exact track of where the alignment padding goes.
236
237 // Set default. Thumb1 function is 1-byte aligned, ARM and Thumb2 are 2-byte
238 // aligned.
239 AFI->setAlign(isThumb1 ? 1U : 2U);
Dale Johannesen56c42ef2007-04-23 20:09:04 +0000240
Evan Chenga8e29892007-01-19 07:51:42 +0000241 // Perform the initial placement of the constant pool entries. To start with,
242 // we put them all at the end of the function.
Evan Chenge03cff62007-02-09 23:59:14 +0000243 std::vector<MachineInstr*> CPEMIs;
Dale Johannesen56c42ef2007-04-23 20:09:04 +0000244 if (!MCP.isEmpty()) {
Evan Cheng7755fac2007-01-26 01:04:44 +0000245 DoInitialPlacement(Fn, CPEMIs);
Evan Chengd3d9d662009-07-23 18:27:47 +0000246 if (isThumb1)
Dale Johannesen56c42ef2007-04-23 20:09:04 +0000247 AFI->setAlign(2U);
248 }
Bob Wilson84945262009-05-12 17:09:30 +0000249
Evan Chenga8e29892007-01-19 07:51:42 +0000250 /// The next UID to take is the first unused one.
Evan Chengf1bbb952008-11-08 00:51:41 +0000251 AFI->initConstPoolEntryUId(CPEMIs.size());
Bob Wilson84945262009-05-12 17:09:30 +0000252
Evan Chenga8e29892007-01-19 07:51:42 +0000253 // Do the initial scan of the function, building up information about the
254 // sizes of each block, the location of all the water, and finding all of the
255 // constant pool users.
256 InitialFunctionScan(Fn, CPEMIs);
257 CPEMIs.clear();
Bob Wilson84945262009-05-12 17:09:30 +0000258
Evan Chenged884f32007-04-03 23:39:48 +0000259 /// Remove dead constant pool entries.
260 RemoveUnusedCPEntries();
261
Evan Chengd1b2c1e2007-01-30 01:18:38 +0000262 // Iteratively place constant pool entries and fix up branches until there
263 // is no change.
264 bool MadeChange = false;
265 while (true) {
266 bool Change = false;
Evan Chenga8e29892007-01-19 07:51:42 +0000267 for (unsigned i = 0, e = CPUsers.size(); i != e; ++i)
Dale Johannesenf1b214d2007-02-28 18:41:23 +0000268 Change |= HandleConstantPoolUser(Fn, i);
Evan Cheng82020102007-07-10 22:00:16 +0000269 DEBUG(dumpBBs());
Evan Chengaf5cbcb2007-01-25 03:12:46 +0000270 for (unsigned i = 0, e = ImmBranches.size(); i != e; ++i)
Evan Chengd1b2c1e2007-01-30 01:18:38 +0000271 Change |= FixUpImmediateBr(Fn, ImmBranches[i]);
Evan Cheng82020102007-07-10 22:00:16 +0000272 DEBUG(dumpBBs());
Evan Chengd1b2c1e2007-01-30 01:18:38 +0000273 if (!Change)
274 break;
275 MadeChange = true;
276 }
Evan Chenged884f32007-04-03 23:39:48 +0000277
Dale Johannesen8593e412007-04-29 19:19:30 +0000278 // After a while, this might be made debug-only, but it is not expensive.
279 verify(Fn);
280
Evan Chengd1b2c1e2007-01-30 01:18:38 +0000281 // If LR has been forced spilled and no far jumps (i.e. BL) has been issued.
282 // Undo the spill / restore of LR if possible.
Evan Chengf49407b2007-03-01 08:26:31 +0000283 if (!HasFarJump && AFI->isLRSpilledForFarJump() && isThumb)
Evan Chengd1b2c1e2007-01-30 01:18:38 +0000284 MadeChange |= UndoLRSpillRestore();
285
Evan Chenga8e29892007-01-19 07:51:42 +0000286 BBSizes.clear();
Dale Johannesen99c49a42007-02-25 00:47:03 +0000287 BBOffsets.clear();
Evan Chenga8e29892007-01-19 07:51:42 +0000288 WaterList.clear();
289 CPUsers.clear();
Evan Chengc99ef082007-02-09 20:54:44 +0000290 CPEntries.clear();
Evan Chengaf5cbcb2007-01-25 03:12:46 +0000291 ImmBranches.clear();
Evan Chengc99ef082007-02-09 20:54:44 +0000292 PushPopMIs.clear();
Evan Chengd1b2c1e2007-01-30 01:18:38 +0000293
294 return MadeChange;
Evan Chenga8e29892007-01-19 07:51:42 +0000295}
296
297/// DoInitialPlacement - Perform the initial placement of the constant pool
298/// entries. To start with, we put them all at the end of the function.
299void ARMConstantIslands::DoInitialPlacement(MachineFunction &Fn,
Bob Wilson84945262009-05-12 17:09:30 +0000300 std::vector<MachineInstr*> &CPEMIs) {
Evan Chenga8e29892007-01-19 07:51:42 +0000301 // Create the basic block to hold the CPE's.
Dan Gohman8e5f2c62008-07-07 23:14:23 +0000302 MachineBasicBlock *BB = Fn.CreateMachineBasicBlock();
303 Fn.push_back(BB);
Bob Wilson84945262009-05-12 17:09:30 +0000304
Evan Chenga8e29892007-01-19 07:51:42 +0000305 // Add all of the constants from the constant pool to the end block, use an
306 // identity mapping of CPI's to CPE's.
307 const std::vector<MachineConstantPoolEntry> &CPs =
308 Fn.getConstantPool()->getConstants();
Bob Wilson84945262009-05-12 17:09:30 +0000309
Evan Chenga8e29892007-01-19 07:51:42 +0000310 const TargetData &TD = *Fn.getTarget().getTargetData();
311 for (unsigned i = 0, e = CPs.size(); i != e; ++i) {
Duncan Sands777d2302009-05-09 07:06:46 +0000312 unsigned Size = TD.getTypeAllocSize(CPs[i].getType());
Evan Chenga8e29892007-01-19 07:51:42 +0000313 // Verify that all constant pool entries are a multiple of 4 bytes. If not,
314 // we would have to pad them out or something so that instructions stay
315 // aligned.
316 assert((Size & 3) == 0 && "CP Entry not multiple of 4 bytes!");
317 MachineInstr *CPEMI =
Dale Johannesenb6728402009-02-13 02:25:56 +0000318 BuildMI(BB, DebugLoc::getUnknownLoc(), TII->get(ARM::CONSTPOOL_ENTRY))
Evan Chenga8e29892007-01-19 07:51:42 +0000319 .addImm(i).addConstantPoolIndex(i).addImm(Size);
320 CPEMIs.push_back(CPEMI);
Evan Chengc99ef082007-02-09 20:54:44 +0000321
322 // Add a new CPEntry, but no corresponding CPUser yet.
323 std::vector<CPEntry> CPEs;
324 CPEs.push_back(CPEntry(CPEMI, i));
325 CPEntries.push_back(CPEs);
326 NumCPEs++;
327 DOUT << "Moved CPI#" << i << " to end of function as #" << i << "\n";
Evan Chenga8e29892007-01-19 07:51:42 +0000328 }
329}
330
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000331/// BBHasFallthrough - Return true if the specified basic block can fallthrough
Evan Chenga8e29892007-01-19 07:51:42 +0000332/// into the block immediately after it.
333static bool BBHasFallthrough(MachineBasicBlock *MBB) {
334 // Get the next machine basic block in the function.
335 MachineFunction::iterator MBBI = MBB;
336 if (next(MBBI) == MBB->getParent()->end()) // Can't fall off end of function.
337 return false;
Bob Wilson84945262009-05-12 17:09:30 +0000338
Evan Chenga8e29892007-01-19 07:51:42 +0000339 MachineBasicBlock *NextBB = next(MBBI);
340 for (MachineBasicBlock::succ_iterator I = MBB->succ_begin(),
341 E = MBB->succ_end(); I != E; ++I)
342 if (*I == NextBB)
343 return true;
Bob Wilson84945262009-05-12 17:09:30 +0000344
Evan Chenga8e29892007-01-19 07:51:42 +0000345 return false;
346}
347
Evan Chengc99ef082007-02-09 20:54:44 +0000348/// findConstPoolEntry - Given the constpool index and CONSTPOOL_ENTRY MI,
349/// look up the corresponding CPEntry.
350ARMConstantIslands::CPEntry
351*ARMConstantIslands::findConstPoolEntry(unsigned CPI,
352 const MachineInstr *CPEMI) {
353 std::vector<CPEntry> &CPEs = CPEntries[CPI];
354 // Number of entries per constpool index should be small, just do a
355 // linear search.
356 for (unsigned i = 0, e = CPEs.size(); i != e; ++i) {
357 if (CPEs[i].CPEMI == CPEMI)
358 return &CPEs[i];
359 }
360 return NULL;
361}
362
Evan Chenga8e29892007-01-19 07:51:42 +0000363/// InitialFunctionScan - Do the initial scan of the function, building up
364/// information about the sizes of each block, the location of all the water,
365/// and finding all of the constant pool users.
366void ARMConstantIslands::InitialFunctionScan(MachineFunction &Fn,
Evan Chenge03cff62007-02-09 23:59:14 +0000367 const std::vector<MachineInstr*> &CPEMIs) {
Dale Johannesen99c49a42007-02-25 00:47:03 +0000368 unsigned Offset = 0;
Evan Chenga8e29892007-01-19 07:51:42 +0000369 for (MachineFunction::iterator MBBI = Fn.begin(), E = Fn.end();
370 MBBI != E; ++MBBI) {
371 MachineBasicBlock &MBB = *MBBI;
Bob Wilson84945262009-05-12 17:09:30 +0000372
Evan Chenga8e29892007-01-19 07:51:42 +0000373 // If this block doesn't fall through into the next MBB, then this is
374 // 'water' that a constant pool island could be placed.
375 if (!BBHasFallthrough(&MBB))
376 WaterList.push_back(&MBB);
Bob Wilson84945262009-05-12 17:09:30 +0000377
Evan Chenga8e29892007-01-19 07:51:42 +0000378 unsigned MBBSize = 0;
379 for (MachineBasicBlock::iterator I = MBB.begin(), E = MBB.end();
380 I != E; ++I) {
381 // Add instruction size to MBBSize.
Nicolas Geoffray52e724a2008-04-16 20:10:13 +0000382 MBBSize += TII->GetInstSizeInBytes(I);
Evan Chenga8e29892007-01-19 07:51:42 +0000383
Evan Chengaf5cbcb2007-01-25 03:12:46 +0000384 int Opc = I->getOpcode();
Chris Lattner749c6f62008-01-07 07:27:27 +0000385 if (I->getDesc().isBranch()) {
Evan Chengaf5cbcb2007-01-25 03:12:46 +0000386 bool isCond = false;
387 unsigned Bits = 0;
388 unsigned Scale = 1;
389 int UOpc = Opc;
390 switch (Opc) {
Dale Johannesen8593e412007-04-29 19:19:30 +0000391 case ARM::tBR_JTr:
Evan Cheng66ac5312009-07-25 00:33:29 +0000392 // A Thumb1 table jump may involve padding; for the offsets to
Dale Johannesen8593e412007-04-29 19:19:30 +0000393 // be right, functions containing these must be 4-byte aligned.
394 AFI->setAlign(2U);
395 if ((Offset+MBBSize)%4 != 0)
396 MBBSize += 2; // padding
397 continue; // Does not get an entry in ImmBranches
Evan Cheng743fa032007-01-25 19:43:52 +0000398 default:
Dale Johannesen8593e412007-04-29 19:19:30 +0000399 continue; // Ignore other JT branches
Evan Chengaf5cbcb2007-01-25 03:12:46 +0000400 case ARM::Bcc:
401 isCond = true;
402 UOpc = ARM::B;
403 // Fallthrough
404 case ARM::B:
405 Bits = 24;
406 Scale = 4;
407 break;
408 case ARM::tBcc:
409 isCond = true;
410 UOpc = ARM::tB;
411 Bits = 8;
412 Scale = 2;
413 break;
414 case ARM::tB:
415 Bits = 11;
416 Scale = 2;
417 break;
David Goodwin5e47a9a2009-06-30 18:04:13 +0000418 case ARM::t2Bcc:
419 isCond = true;
420 UOpc = ARM::t2B;
421 Bits = 20;
422 Scale = 2;
423 break;
424 case ARM::t2B:
425 Bits = 24;
426 Scale = 2;
427 break;
Evan Chengaf5cbcb2007-01-25 03:12:46 +0000428 }
Evan Chengb43216e2007-02-01 10:16:15 +0000429
430 // Record this immediate branch.
Evan Chengbd5d3db2007-02-03 02:08:34 +0000431 unsigned MaxOffs = ((1 << (Bits-1))-1) * Scale;
Evan Chengb43216e2007-02-01 10:16:15 +0000432 ImmBranches.push_back(ImmBranch(I, MaxOffs, isCond, UOpc));
Evan Chengaf5cbcb2007-01-25 03:12:46 +0000433 }
434
Evan Chengd1b2c1e2007-01-30 01:18:38 +0000435 if (Opc == ARM::tPUSH || Opc == ARM::tPOP_RET)
436 PushPopMIs.push_back(I);
437
Evan Chengd3d9d662009-07-23 18:27:47 +0000438 if (Opc == ARM::CONSTPOOL_ENTRY)
439 continue;
440
Evan Chenga8e29892007-01-19 07:51:42 +0000441 // Scan the instructions for constant pool operands.
442 for (unsigned op = 0, e = I->getNumOperands(); op != e; ++op)
Dan Gohmand735b802008-10-03 15:45:36 +0000443 if (I->getOperand(op).isCPI()) {
Evan Chenga8e29892007-01-19 07:51:42 +0000444 // We found one. The addressing mode tells us the max displacement
445 // from the PC that this instruction permits.
Bob Wilson84945262009-05-12 17:09:30 +0000446
Evan Chenga8e29892007-01-19 07:51:42 +0000447 // Basic size info comes from the TSFlags field.
Evan Chengb43216e2007-02-01 10:16:15 +0000448 unsigned Bits = 0;
449 unsigned Scale = 1;
Evan Cheng5d8f1ca2009-07-21 23:56:01 +0000450 bool NegOk = false;
Evan Chengd3d9d662009-07-23 18:27:47 +0000451 bool IsSoImm = false;
452
453 switch (Opc) {
Bob Wilson84945262009-05-12 17:09:30 +0000454 default:
Torok Edwinc23197a2009-07-14 16:55:14 +0000455 llvm_unreachable("Unknown addressing mode for CP reference!");
Evan Chengd3d9d662009-07-23 18:27:47 +0000456 break;
457
458 // Taking the address of a CP entry.
459 case ARM::LEApcrel:
460 // This takes a SoImm, which is 8 bit immediate rotated. We'll
461 // pretend the maximum offset is 255 * 4. Since each instruction
462 // 4 byte wide, this is always correct. We'llc heck for other
463 // displacements that fits in a SoImm as well.
Evan Chengb43216e2007-02-01 10:16:15 +0000464 Bits = 8;
Evan Chengd3d9d662009-07-23 18:27:47 +0000465 Scale = 4;
466 NegOk = true;
467 IsSoImm = true;
468 break;
469 case ARM::t2LEApcrel:
470 Bits = 12;
Evan Cheng5d8f1ca2009-07-21 23:56:01 +0000471 NegOk = true;
Evan Chenga8e29892007-01-19 07:51:42 +0000472 break;
Evan Chengd3d9d662009-07-23 18:27:47 +0000473 case ARM::tLEApcrel:
474 Bits = 8;
475 Scale = 4;
476 break;
477
478 case ARM::LDR:
479 case ARM::LDRcp:
480 case ARM::t2LDRpci:
Evan Cheng556f33c2007-02-01 20:44:52 +0000481 Bits = 12; // +-offset_12
Evan Cheng5d8f1ca2009-07-21 23:56:01 +0000482 NegOk = true;
Evan Chenga8e29892007-01-19 07:51:42 +0000483 break;
Evan Chengd3d9d662009-07-23 18:27:47 +0000484
485 case ARM::tLDRpci:
486 case ARM::tLDRcp:
Evan Chengb43216e2007-02-01 10:16:15 +0000487 Bits = 8;
488 Scale = 4; // +(offset_8*4)
Evan Cheng012f2d92007-01-24 08:53:17 +0000489 break;
Evan Chengd3d9d662009-07-23 18:27:47 +0000490
491 case ARM::FLDD:
492 case ARM::FLDS:
493 Bits = 8;
494 Scale = 4; // +-(offset_8*4)
495 NegOk = true;
Evan Cheng055b0312009-06-29 07:51:04 +0000496 break;
Evan Chenga8e29892007-01-19 07:51:42 +0000497 }
Evan Chengb43216e2007-02-01 10:16:15 +0000498
Evan Chenga8e29892007-01-19 07:51:42 +0000499 // Remember that this is a user of a CP entry.
Chris Lattner8aa797a2007-12-30 23:10:15 +0000500 unsigned CPI = I->getOperand(op).getIndex();
Evan Chengc99ef082007-02-09 20:54:44 +0000501 MachineInstr *CPEMI = CPEMIs[CPI];
Bob Wilson84945262009-05-12 17:09:30 +0000502 unsigned MaxOffs = ((1 << Bits)-1) * Scale;
Evan Chengd3d9d662009-07-23 18:27:47 +0000503 CPUsers.push_back(CPUser(I, CPEMI, MaxOffs, NegOk, IsSoImm));
Evan Chengc99ef082007-02-09 20:54:44 +0000504
505 // Increment corresponding CPEntry reference count.
506 CPEntry *CPE = findConstPoolEntry(CPI, CPEMI);
507 assert(CPE && "Cannot find a corresponding CPEntry!");
508 CPE->RefCount++;
Bob Wilson84945262009-05-12 17:09:30 +0000509
Evan Chenga8e29892007-01-19 07:51:42 +0000510 // Instructions can only use one CP entry, don't bother scanning the
511 // rest of the operands.
512 break;
513 }
514 }
Evan Cheng2021abe2007-02-01 01:09:47 +0000515
Dale Johannesen8593e412007-04-29 19:19:30 +0000516 // In thumb mode, if this block is a constpool island, we may need padding
517 // so it's aligned on 4 byte boundary.
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000518 if (isThumb &&
Evan Cheng05cc4242007-02-02 19:09:19 +0000519 !MBB.empty() &&
Dale Johannesen8593e412007-04-29 19:19:30 +0000520 MBB.begin()->getOpcode() == ARM::CONSTPOOL_ENTRY &&
521 (Offset%4) != 0)
Evan Cheng2021abe2007-02-01 01:09:47 +0000522 MBBSize += 2;
523
Evan Chenga8e29892007-01-19 07:51:42 +0000524 BBSizes.push_back(MBBSize);
Dale Johannesen99c49a42007-02-25 00:47:03 +0000525 BBOffsets.push_back(Offset);
526 Offset += MBBSize;
Evan Chenga8e29892007-01-19 07:51:42 +0000527 }
528}
529
Evan Chenga8e29892007-01-19 07:51:42 +0000530/// GetOffsetOf - Return the current offset of the specified machine instruction
531/// from the start of the function. This offset changes as stuff is moved
532/// around inside the function.
533unsigned ARMConstantIslands::GetOffsetOf(MachineInstr *MI) const {
534 MachineBasicBlock *MBB = MI->getParent();
Bob Wilson84945262009-05-12 17:09:30 +0000535
Evan Chenga8e29892007-01-19 07:51:42 +0000536 // The offset is composed of two things: the sum of the sizes of all MBB's
537 // before this instruction's block, and the offset from the start of the block
538 // it is in.
Dale Johannesen99c49a42007-02-25 00:47:03 +0000539 unsigned Offset = BBOffsets[MBB->getNumber()];
Evan Chenga8e29892007-01-19 07:51:42 +0000540
Dale Johannesen8593e412007-04-29 19:19:30 +0000541 // If we're looking for a CONSTPOOL_ENTRY in Thumb, see if this block has
542 // alignment padding, and compensate if so.
Bob Wilson84945262009-05-12 17:09:30 +0000543 if (isThumb &&
544 MI->getOpcode() == ARM::CONSTPOOL_ENTRY &&
Dale Johannesen8593e412007-04-29 19:19:30 +0000545 Offset%4 != 0)
546 Offset += 2;
547
Evan Chenga8e29892007-01-19 07:51:42 +0000548 // Sum instructions before MI in MBB.
549 for (MachineBasicBlock::iterator I = MBB->begin(); ; ++I) {
550 assert(I != MBB->end() && "Didn't find MI in its own basic block?");
551 if (&*I == MI) return Offset;
Nicolas Geoffray52e724a2008-04-16 20:10:13 +0000552 Offset += TII->GetInstSizeInBytes(I);
Evan Chenga8e29892007-01-19 07:51:42 +0000553 }
554}
555
556/// CompareMBBNumbers - Little predicate function to sort the WaterList by MBB
557/// ID.
558static bool CompareMBBNumbers(const MachineBasicBlock *LHS,
559 const MachineBasicBlock *RHS) {
560 return LHS->getNumber() < RHS->getNumber();
561}
562
563/// UpdateForInsertedWaterBlock - When a block is newly inserted into the
564/// machine function, it upsets all of the block numbers. Renumber the blocks
565/// and update the arrays that parallel this numbering.
566void ARMConstantIslands::UpdateForInsertedWaterBlock(MachineBasicBlock *NewBB) {
567 // Renumber the MBB's to keep them consequtive.
568 NewBB->getParent()->RenumberBlocks(NewBB);
Bob Wilson84945262009-05-12 17:09:30 +0000569
Evan Chenga8e29892007-01-19 07:51:42 +0000570 // Insert a size into BBSizes to align it properly with the (newly
571 // renumbered) block numbers.
572 BBSizes.insert(BBSizes.begin()+NewBB->getNumber(), 0);
Dale Johannesen99c49a42007-02-25 00:47:03 +0000573
574 // Likewise for BBOffsets.
575 BBOffsets.insert(BBOffsets.begin()+NewBB->getNumber(), 0);
Bob Wilson84945262009-05-12 17:09:30 +0000576
577 // Next, update WaterList. Specifically, we need to add NewMBB as having
Evan Chenga8e29892007-01-19 07:51:42 +0000578 // available water after it.
Evan Chenge03cff62007-02-09 23:59:14 +0000579 std::vector<MachineBasicBlock*>::iterator IP =
Evan Chenga8e29892007-01-19 07:51:42 +0000580 std::lower_bound(WaterList.begin(), WaterList.end(), NewBB,
581 CompareMBBNumbers);
582 WaterList.insert(IP, NewBB);
583}
584
585
586/// Split the basic block containing MI into two blocks, which are joined by
587/// an unconditional branch. Update datastructures and renumber blocks to
Evan Cheng0c615842007-01-31 02:22:22 +0000588/// account for this change and returns the newly created block.
589MachineBasicBlock *ARMConstantIslands::SplitBlockBeforeInstr(MachineInstr *MI) {
Evan Chenga8e29892007-01-19 07:51:42 +0000590 MachineBasicBlock *OrigBB = MI->getParent();
Dan Gohman8e5f2c62008-07-07 23:14:23 +0000591 MachineFunction &MF = *OrigBB->getParent();
Evan Chenga8e29892007-01-19 07:51:42 +0000592
593 // Create a new MBB for the code after the OrigBB.
Bob Wilson84945262009-05-12 17:09:30 +0000594 MachineBasicBlock *NewBB =
595 MF.CreateMachineBasicBlock(OrigBB->getBasicBlock());
Evan Chenga8e29892007-01-19 07:51:42 +0000596 MachineFunction::iterator MBBI = OrigBB; ++MBBI;
Dan Gohman8e5f2c62008-07-07 23:14:23 +0000597 MF.insert(MBBI, NewBB);
Bob Wilson84945262009-05-12 17:09:30 +0000598
Evan Chenga8e29892007-01-19 07:51:42 +0000599 // Splice the instructions starting with MI over to NewBB.
600 NewBB->splice(NewBB->end(), OrigBB, MI, OrigBB->end());
Bob Wilson84945262009-05-12 17:09:30 +0000601
Evan Chenga8e29892007-01-19 07:51:42 +0000602 // Add an unconditional branch from OrigBB to NewBB.
Evan Chenga9b8b8d2007-01-31 18:29:27 +0000603 // Note the new unconditional branch is not being recorded.
Dale Johannesenb6728402009-02-13 02:25:56 +0000604 // There doesn't seem to be meaningful DebugInfo available; this doesn't
605 // correspond to anything in the source.
Evan Cheng58541fd2009-07-07 01:16:41 +0000606 unsigned Opc = isThumb ? (isThumb2 ? ARM::t2B : ARM::tB) : ARM::B;
607 BuildMI(OrigBB, DebugLoc::getUnknownLoc(), TII->get(Opc)).addMBB(NewBB);
Evan Chenga8e29892007-01-19 07:51:42 +0000608 NumSplit++;
Bob Wilson84945262009-05-12 17:09:30 +0000609
Evan Chenga8e29892007-01-19 07:51:42 +0000610 // Update the CFG. All succs of OrigBB are now succs of NewBB.
611 while (!OrigBB->succ_empty()) {
612 MachineBasicBlock *Succ = *OrigBB->succ_begin();
613 OrigBB->removeSuccessor(Succ);
614 NewBB->addSuccessor(Succ);
Bob Wilson84945262009-05-12 17:09:30 +0000615
Evan Chenga8e29892007-01-19 07:51:42 +0000616 // This pass should be run after register allocation, so there should be no
617 // PHI nodes to update.
618 assert((Succ->empty() || Succ->begin()->getOpcode() != TargetInstrInfo::PHI)
619 && "PHI nodes should be eliminated by now!");
620 }
Bob Wilson84945262009-05-12 17:09:30 +0000621
Evan Chenga8e29892007-01-19 07:51:42 +0000622 // OrigBB branches to NewBB.
623 OrigBB->addSuccessor(NewBB);
Bob Wilson84945262009-05-12 17:09:30 +0000624
Evan Chenga8e29892007-01-19 07:51:42 +0000625 // Update internal data structures to account for the newly inserted MBB.
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000626 // This is almost the same as UpdateForInsertedWaterBlock, except that
627 // the Water goes after OrigBB, not NewBB.
Dan Gohman8e5f2c62008-07-07 23:14:23 +0000628 MF.RenumberBlocks(NewBB);
Bob Wilson84945262009-05-12 17:09:30 +0000629
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000630 // Insert a size into BBSizes to align it properly with the (newly
631 // renumbered) block numbers.
632 BBSizes.insert(BBSizes.begin()+NewBB->getNumber(), 0);
Bob Wilson84945262009-05-12 17:09:30 +0000633
Dale Johannesen99c49a42007-02-25 00:47:03 +0000634 // Likewise for BBOffsets.
635 BBOffsets.insert(BBOffsets.begin()+NewBB->getNumber(), 0);
636
Bob Wilson84945262009-05-12 17:09:30 +0000637 // Next, update WaterList. Specifically, we need to add OrigMBB as having
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000638 // available water after it (but not if it's already there, which happens
639 // when splitting before a conditional branch that is followed by an
640 // unconditional branch - in that case we want to insert NewBB).
641 std::vector<MachineBasicBlock*>::iterator IP =
642 std::lower_bound(WaterList.begin(), WaterList.end(), OrigBB,
643 CompareMBBNumbers);
644 MachineBasicBlock* WaterBB = *IP;
645 if (WaterBB == OrigBB)
646 WaterList.insert(next(IP), NewBB);
647 else
648 WaterList.insert(IP, OrigBB);
649
Dale Johannesen8593e412007-04-29 19:19:30 +0000650 // Figure out how large the first NewMBB is. (It cannot
651 // contain a constpool_entry or tablejump.)
Evan Chenga8e29892007-01-19 07:51:42 +0000652 unsigned NewBBSize = 0;
653 for (MachineBasicBlock::iterator I = NewBB->begin(), E = NewBB->end();
654 I != E; ++I)
Nicolas Geoffray52e724a2008-04-16 20:10:13 +0000655 NewBBSize += TII->GetInstSizeInBytes(I);
Bob Wilson84945262009-05-12 17:09:30 +0000656
Dale Johannesen99c49a42007-02-25 00:47:03 +0000657 unsigned OrigBBI = OrigBB->getNumber();
658 unsigned NewBBI = NewBB->getNumber();
Evan Chenga8e29892007-01-19 07:51:42 +0000659 // Set the size of NewBB in BBSizes.
Dale Johannesen99c49a42007-02-25 00:47:03 +0000660 BBSizes[NewBBI] = NewBBSize;
Bob Wilson84945262009-05-12 17:09:30 +0000661
Evan Chenga8e29892007-01-19 07:51:42 +0000662 // We removed instructions from UserMBB, subtract that off from its size.
Evan Chengaf5cbcb2007-01-25 03:12:46 +0000663 // Add 2 or 4 to the block to count the unconditional branch we added to it.
Evan Chengd3d9d662009-07-23 18:27:47 +0000664 unsigned delta = isThumb1 ? 2 : 4;
Dale Johannesen99c49a42007-02-25 00:47:03 +0000665 BBSizes[OrigBBI] -= NewBBSize - delta;
666
667 // ...and adjust BBOffsets for NewBB accordingly.
668 BBOffsets[NewBBI] = BBOffsets[OrigBBI] + BBSizes[OrigBBI];
669
670 // All BBOffsets following these blocks must be modified.
671 AdjustBBOffsetsAfter(NewBB, delta);
Evan Cheng0c615842007-01-31 02:22:22 +0000672
673 return NewBB;
Evan Chenga8e29892007-01-19 07:51:42 +0000674}
675
Dale Johannesen8593e412007-04-29 19:19:30 +0000676/// OffsetIsInRange - Checks whether UserOffset (the location of a constant pool
Bob Wilson84945262009-05-12 17:09:30 +0000677/// reference) is within MaxDisp of TrialOffset (a proposed location of a
Dale Johannesen8593e412007-04-29 19:19:30 +0000678/// constant pool entry).
Bob Wilson84945262009-05-12 17:09:30 +0000679bool ARMConstantIslands::OffsetIsInRange(unsigned UserOffset,
Evan Chengd3d9d662009-07-23 18:27:47 +0000680 unsigned TrialOffset, unsigned MaxDisp,
681 bool NegativeOK, bool IsSoImm) {
Bob Wilson84945262009-05-12 17:09:30 +0000682 // On Thumb offsets==2 mod 4 are rounded down by the hardware for
683 // purposes of the displacement computation; compensate for that here.
Dale Johannesen8593e412007-04-29 19:19:30 +0000684 // Effectively, the valid range of displacements is 2 bytes smaller for such
685 // references.
686 if (isThumb && UserOffset%4 !=0)
687 UserOffset -= 2;
688 // CPEs will be rounded up to a multiple of 4.
689 if (isThumb && TrialOffset%4 != 0)
690 TrialOffset += 2;
691
Dale Johannesen99c49a42007-02-25 00:47:03 +0000692 if (UserOffset <= TrialOffset) {
693 // User before the Trial.
Evan Chengd3d9d662009-07-23 18:27:47 +0000694 if (TrialOffset - UserOffset <= MaxDisp)
695 return true;
Evan Cheng40efc252009-07-24 19:31:03 +0000696 // FIXME: Make use full range of soimm values.
Dale Johannesen99c49a42007-02-25 00:47:03 +0000697 } else if (NegativeOK) {
Evan Chengd3d9d662009-07-23 18:27:47 +0000698 if (UserOffset - TrialOffset <= MaxDisp)
699 return true;
Evan Cheng40efc252009-07-24 19:31:03 +0000700 // FIXME: Make use full range of soimm values.
Dale Johannesen99c49a42007-02-25 00:47:03 +0000701 }
702 return false;
703}
704
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000705/// WaterIsInRange - Returns true if a CPE placed after the specified
706/// Water (a basic block) will be in range for the specific MI.
707
708bool ARMConstantIslands::WaterIsInRange(unsigned UserOffset,
Evan Cheng5d8f1ca2009-07-21 23:56:01 +0000709 MachineBasicBlock* Water, CPUser &U) {
Dale Johannesen5d9c4b62007-07-11 18:32:38 +0000710 unsigned MaxDisp = U.MaxDisp;
Bob Wilson84945262009-05-12 17:09:30 +0000711 unsigned CPEOffset = BBOffsets[Water->getNumber()] +
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000712 BBSizes[Water->getNumber()];
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000713
Dale Johannesend959aa42007-04-02 20:31:06 +0000714 // If the CPE is to be inserted before the instruction, that will raise
Dale Johannesen8593e412007-04-29 19:19:30 +0000715 // the offset of the instruction. (Currently applies only to ARM, so
716 // no alignment compensation attempted here.)
Dale Johannesend959aa42007-04-02 20:31:06 +0000717 if (CPEOffset < UserOffset)
Dale Johannesen5d9c4b62007-07-11 18:32:38 +0000718 UserOffset += U.CPEMI->getOperand(2).getImm();
Dale Johannesend959aa42007-04-02 20:31:06 +0000719
Evan Chengd3d9d662009-07-23 18:27:47 +0000720 return OffsetIsInRange(UserOffset, CPEOffset, MaxDisp, U.NegOk, U.IsSoImm);
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000721}
722
723/// CPEIsInRange - Returns true if the distance between specific MI and
Evan Chengc0dbec72007-01-31 19:57:44 +0000724/// specific ConstPool entry instruction can fit in MI's displacement field.
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000725bool ARMConstantIslands::CPEIsInRange(MachineInstr *MI, unsigned UserOffset,
Evan Cheng5d8f1ca2009-07-21 23:56:01 +0000726 MachineInstr *CPEMI, unsigned MaxDisp,
727 bool NegOk, bool DoDump) {
Dale Johannesen8593e412007-04-29 19:19:30 +0000728 unsigned CPEOffset = GetOffsetOf(CPEMI);
729 assert(CPEOffset%4 == 0 && "Misaligned CPE");
Evan Cheng2021abe2007-02-01 01:09:47 +0000730
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000731 if (DoDump) {
732 DOUT << "User of CPE#" << CPEMI->getOperand(0).getImm()
733 << " max delta=" << MaxDisp
734 << " insn address=" << UserOffset
735 << " CPE address=" << CPEOffset
736 << " offset=" << int(CPEOffset-UserOffset) << "\t" << *MI;
737 }
Evan Chengc0dbec72007-01-31 19:57:44 +0000738
Evan Cheng5d8f1ca2009-07-21 23:56:01 +0000739 return OffsetIsInRange(UserOffset, CPEOffset, MaxDisp, NegOk);
Evan Chengc0dbec72007-01-31 19:57:44 +0000740}
741
Evan Chengd1e7d9a2009-01-28 00:53:34 +0000742#ifndef NDEBUG
Evan Chengc99ef082007-02-09 20:54:44 +0000743/// BBIsJumpedOver - Return true of the specified basic block's only predecessor
744/// unconditionally branches to its only successor.
745static bool BBIsJumpedOver(MachineBasicBlock *MBB) {
746 if (MBB->pred_size() != 1 || MBB->succ_size() != 1)
747 return false;
748
749 MachineBasicBlock *Succ = *MBB->succ_begin();
750 MachineBasicBlock *Pred = *MBB->pred_begin();
751 MachineInstr *PredMI = &Pred->back();
David Goodwin5e47a9a2009-06-30 18:04:13 +0000752 if (PredMI->getOpcode() == ARM::B || PredMI->getOpcode() == ARM::tB
753 || PredMI->getOpcode() == ARM::t2B)
Evan Chengc99ef082007-02-09 20:54:44 +0000754 return PredMI->getOperand(0).getMBB() == Succ;
755 return false;
756}
Evan Chengd1e7d9a2009-01-28 00:53:34 +0000757#endif // NDEBUG
Evan Chengc99ef082007-02-09 20:54:44 +0000758
Bob Wilson84945262009-05-12 17:09:30 +0000759void ARMConstantIslands::AdjustBBOffsetsAfter(MachineBasicBlock *BB,
Dale Johannesen8593e412007-04-29 19:19:30 +0000760 int delta) {
761 MachineFunction::iterator MBBI = BB; MBBI = next(MBBI);
Evan Chengd3d9d662009-07-23 18:27:47 +0000762 for(unsigned i = BB->getNumber()+1, e = BB->getParent()->getNumBlockIDs();
763 i < e; ++i) {
Dale Johannesen99c49a42007-02-25 00:47:03 +0000764 BBOffsets[i] += delta;
Dale Johannesen8593e412007-04-29 19:19:30 +0000765 // If some existing blocks have padding, adjust the padding as needed, a
766 // bit tricky. delta can be negative so don't use % on that.
Evan Chengd3d9d662009-07-23 18:27:47 +0000767 if (!isThumb)
768 continue;
769 MachineBasicBlock *MBB = MBBI;
770 if (!MBB->empty()) {
771 // Constant pool entries require padding.
772 if (MBB->begin()->getOpcode() == ARM::CONSTPOOL_ENTRY) {
773 unsigned oldOffset = BBOffsets[i] - delta;
774 if (oldOffset%4==0 && BBOffsets[i]%4!=0) {
775 // add new padding
776 BBSizes[i] += 2;
777 delta += 2;
778 } else if (oldOffset%4!=0 && BBOffsets[i]%4==0) {
779 // remove existing padding
780 BBSizes[i] -=2;
781 delta -= 2;
Dale Johannesen8593e412007-04-29 19:19:30 +0000782 }
Dale Johannesen8593e412007-04-29 19:19:30 +0000783 }
Evan Chengd3d9d662009-07-23 18:27:47 +0000784 // Thumb1 jump tables require padding. They should be at the end;
785 // following unconditional branches are removed by AnalyzeBranch.
Evan Cheng78947622009-07-24 18:20:44 +0000786 MachineInstr *ThumbJTMI = prior(MBB->end());
Evan Cheng66ac5312009-07-25 00:33:29 +0000787 if (ThumbJTMI->getOpcode() == ARM::tBR_JTr) {
Evan Chengd3d9d662009-07-23 18:27:47 +0000788 unsigned newMIOffset = GetOffsetOf(ThumbJTMI);
789 unsigned oldMIOffset = newMIOffset - delta;
790 if (oldMIOffset%4 == 0 && newMIOffset%4 != 0) {
791 // remove existing padding
792 BBSizes[i] -= 2;
793 delta -= 2;
794 } else if (oldMIOffset%4 != 0 && newMIOffset%4 == 0) {
795 // add new padding
796 BBSizes[i] += 2;
797 delta += 2;
798 }
799 }
800 if (delta==0)
801 return;
Dale Johannesen8593e412007-04-29 19:19:30 +0000802 }
Evan Chengd3d9d662009-07-23 18:27:47 +0000803 MBBI = next(MBBI);
Dale Johannesen8593e412007-04-29 19:19:30 +0000804 }
Dale Johannesen99c49a42007-02-25 00:47:03 +0000805}
806
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000807/// DecrementOldEntry - find the constant pool entry with index CPI
808/// and instruction CPEMI, and decrement its refcount. If the refcount
Bob Wilson84945262009-05-12 17:09:30 +0000809/// becomes 0 remove the entry and instruction. Returns true if we removed
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000810/// the entry, false if we didn't.
Evan Chenga8e29892007-01-19 07:51:42 +0000811
Evan Chenged884f32007-04-03 23:39:48 +0000812bool ARMConstantIslands::DecrementOldEntry(unsigned CPI, MachineInstr *CPEMI) {
Evan Chengc99ef082007-02-09 20:54:44 +0000813 // Find the old entry. Eliminate it if it is no longer used.
Evan Chenged884f32007-04-03 23:39:48 +0000814 CPEntry *CPE = findConstPoolEntry(CPI, CPEMI);
815 assert(CPE && "Unexpected!");
816 if (--CPE->RefCount == 0) {
817 RemoveDeadCPEMI(CPEMI);
818 CPE->CPEMI = NULL;
Evan Chengc99ef082007-02-09 20:54:44 +0000819 NumCPEs--;
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000820 return true;
821 }
822 return false;
823}
824
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000825/// LookForCPEntryInRange - see if the currently referenced CPE is in range;
826/// if not, see if an in-range clone of the CPE is in range, and if so,
827/// change the data structures so the user references the clone. Returns:
828/// 0 = no existing entry found
829/// 1 = entry found, and there were no code insertions or deletions
830/// 2 = entry found, and there were code insertions or deletions
831int ARMConstantIslands::LookForExistingCPEntry(CPUser& U, unsigned UserOffset)
832{
833 MachineInstr *UserMI = U.MI;
834 MachineInstr *CPEMI = U.CPEMI;
835
836 // Check to see if the CPE is already in-range.
Evan Cheng5d8f1ca2009-07-21 23:56:01 +0000837 if (CPEIsInRange(UserMI, UserOffset, CPEMI, U.MaxDisp, U.NegOk, true)) {
Evan Cheng82020102007-07-10 22:00:16 +0000838 DOUT << "In range\n";
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000839 return 1;
Evan Chengc99ef082007-02-09 20:54:44 +0000840 }
841
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000842 // No. Look for previously created clones of the CPE that are in range.
Chris Lattner8aa797a2007-12-30 23:10:15 +0000843 unsigned CPI = CPEMI->getOperand(1).getIndex();
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000844 std::vector<CPEntry> &CPEs = CPEntries[CPI];
845 for (unsigned i = 0, e = CPEs.size(); i != e; ++i) {
846 // We already tried this one
847 if (CPEs[i].CPEMI == CPEMI)
848 continue;
849 // Removing CPEs can leave empty entries, skip
850 if (CPEs[i].CPEMI == NULL)
851 continue;
Evan Cheng5d8f1ca2009-07-21 23:56:01 +0000852 if (CPEIsInRange(UserMI, UserOffset, CPEs[i].CPEMI, U.MaxDisp, U.NegOk)) {
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000853 DOUT << "Replacing CPE#" << CPI << " with CPE#" << CPEs[i].CPI << "\n";
854 // Point the CPUser node to the replacement
855 U.CPEMI = CPEs[i].CPEMI;
856 // Change the CPI in the instruction operand to refer to the clone.
857 for (unsigned j = 0, e = UserMI->getNumOperands(); j != e; ++j)
Dan Gohmand735b802008-10-03 15:45:36 +0000858 if (UserMI->getOperand(j).isCPI()) {
Chris Lattner8aa797a2007-12-30 23:10:15 +0000859 UserMI->getOperand(j).setIndex(CPEs[i].CPI);
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000860 break;
861 }
862 // Adjust the refcount of the clone...
863 CPEs[i].RefCount++;
864 // ...and the original. If we didn't remove the old entry, none of the
865 // addresses changed, so we don't need another pass.
Evan Chenged884f32007-04-03 23:39:48 +0000866 return DecrementOldEntry(CPI, CPEMI) ? 2 : 1;
Dale Johannesen88e37ae2007-02-23 05:02:36 +0000867 }
868 }
869 return 0;
870}
871
Dale Johannesenf1b214d2007-02-28 18:41:23 +0000872/// getUnconditionalBrDisp - Returns the maximum displacement that can fit in
873/// the specific unconditional branch instruction.
874static inline unsigned getUnconditionalBrDisp(int Opc) {
David Goodwin5e47a9a2009-06-30 18:04:13 +0000875 switch (Opc) {
876 case ARM::tB:
877 return ((1<<10)-1)*2;
878 case ARM::t2B:
879 return ((1<<23)-1)*2;
880 default:
881 break;
882 }
883
884 return ((1<<23)-1)*4;
Dale Johannesenf1b214d2007-02-28 18:41:23 +0000885}
886
Dale Johannesen8593e412007-04-29 19:19:30 +0000887/// AcceptWater - Small amount of common code factored out of the following.
888
Bob Wilson84945262009-05-12 17:09:30 +0000889MachineBasicBlock* ARMConstantIslands::AcceptWater(MachineBasicBlock *WaterBB,
Dale Johannesen8593e412007-04-29 19:19:30 +0000890 std::vector<MachineBasicBlock*>::iterator IP) {
891 DOUT << "found water in range\n";
892 // Remove the original WaterList entry; we want subsequent
893 // insertions in this vicinity to go after the one we're
894 // about to insert. This considerably reduces the number
895 // of times we have to move the same CPE more than once.
896 WaterList.erase(IP);
897 // CPE goes before following block (NewMBB).
898 return next(MachineFunction::iterator(WaterBB));
899}
900
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000901/// LookForWater - look for an existing entry in the WaterList in which
902/// we can place the CPE referenced from U so it's within range of U's MI.
903/// Returns true if found, false if not. If it returns true, *NewMBB
Dale Johannesen8593e412007-04-29 19:19:30 +0000904/// is set to the WaterList entry.
Evan Chengd3d9d662009-07-23 18:27:47 +0000905/// For ARM, we prefer the water that's farthest away. For Thumb, prefer
Dale Johannesen8593e412007-04-29 19:19:30 +0000906/// water that will not introduce padding to water that will; within each
907/// group, prefer the water that's farthest away.
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000908bool ARMConstantIslands::LookForWater(CPUser &U, unsigned UserOffset,
Dale Johannesen8593e412007-04-29 19:19:30 +0000909 MachineBasicBlock** NewMBB) {
910 std::vector<MachineBasicBlock*>::iterator IPThatWouldPad;
911 MachineBasicBlock* WaterBBThatWouldPad = NULL;
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000912 if (!WaterList.empty()) {
913 for (std::vector<MachineBasicBlock*>::iterator IP = prior(WaterList.end()),
Evan Chengd3d9d662009-07-23 18:27:47 +0000914 B = WaterList.begin();; --IP) {
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000915 MachineBasicBlock* WaterBB = *IP;
Dale Johannesen5d9c4b62007-07-11 18:32:38 +0000916 if (WaterIsInRange(UserOffset, WaterBB, U)) {
Evan Chengd3d9d662009-07-23 18:27:47 +0000917 unsigned WBBId = WaterBB->getNumber();
Dale Johannesen8593e412007-04-29 19:19:30 +0000918 if (isThumb &&
Evan Chengd3d9d662009-07-23 18:27:47 +0000919 (BBOffsets[WBBId] + BBSizes[WBBId])%4 != 0) {
Dale Johannesen8593e412007-04-29 19:19:30 +0000920 // This is valid Water, but would introduce padding. Remember
921 // it in case we don't find any Water that doesn't do this.
922 if (!WaterBBThatWouldPad) {
923 WaterBBThatWouldPad = WaterBB;
924 IPThatWouldPad = IP;
925 }
926 } else {
927 *NewMBB = AcceptWater(WaterBB, IP);
928 return true;
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000929 }
Evan Chengd3d9d662009-07-23 18:27:47 +0000930 }
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000931 if (IP == B)
932 break;
933 }
934 }
Dale Johannesen8593e412007-04-29 19:19:30 +0000935 if (isThumb && WaterBBThatWouldPad) {
936 *NewMBB = AcceptWater(WaterBBThatWouldPad, IPThatWouldPad);
937 return true;
938 }
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000939 return false;
940}
941
Bob Wilson84945262009-05-12 17:09:30 +0000942/// CreateNewWater - No existing WaterList entry will work for
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000943/// CPUsers[CPUserIndex], so create a place to put the CPE. The end of the
944/// block is used if in range, and the conditional branch munged so control
945/// flow is correct. Otherwise the block is split to create a hole with an
946/// unconditional branch around it. In either case *NewMBB is set to a
947/// block following which the new island can be inserted (the WaterList
948/// is not adjusted).
949
Bob Wilson84945262009-05-12 17:09:30 +0000950void ARMConstantIslands::CreateNewWater(unsigned CPUserIndex,
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000951 unsigned UserOffset, MachineBasicBlock** NewMBB) {
952 CPUser &U = CPUsers[CPUserIndex];
953 MachineInstr *UserMI = U.MI;
954 MachineInstr *CPEMI = U.CPEMI;
955 MachineBasicBlock *UserMBB = UserMI->getParent();
Bob Wilson84945262009-05-12 17:09:30 +0000956 unsigned OffsetOfNextBlock = BBOffsets[UserMBB->getNumber()] +
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000957 BBSizes[UserMBB->getNumber()];
Dale Johannesen8593e412007-04-29 19:19:30 +0000958 assert(OffsetOfNextBlock== BBOffsets[UserMBB->getNumber()+1]);
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000959
960 // If the use is at the end of the block, or the end of the block
Dale Johannesen8593e412007-04-29 19:19:30 +0000961 // is within range, make new water there. (The addition below is
Evan Chengd3d9d662009-07-23 18:27:47 +0000962 // for the unconditional branch we will be adding: 4 bytes on ARM + Thumb2,
963 // 2 on Thumb1. Possible Thumb1 alignment padding is allowed for
Dale Johannesen8593e412007-04-29 19:19:30 +0000964 // inside OffsetIsInRange.
Bob Wilson84945262009-05-12 17:09:30 +0000965 // If the block ends in an unconditional branch already, it is water,
Dale Johannesen8593e412007-04-29 19:19:30 +0000966 // and is known to be out of range, so we'll always be adding a branch.)
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000967 if (&UserMBB->back() == UserMI ||
Evan Chengd3d9d662009-07-23 18:27:47 +0000968 OffsetIsInRange(UserOffset, OffsetOfNextBlock + (isThumb1 ? 2: 4),
969 U.MaxDisp, U.NegOk, U.IsSoImm)) {
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000970 DOUT << "Split at end of block\n";
971 if (&UserMBB->back() == UserMI)
972 assert(BBHasFallthrough(UserMBB) && "Expected a fallthrough BB!");
973 *NewMBB = next(MachineFunction::iterator(UserMBB));
974 // Add an unconditional branch from UserMBB to fallthrough block.
975 // Record it for branch lengthening; this new branch will not get out of
976 // range, but if the preceding conditional branch is out of range, the
977 // targets will be exchanged, and the altered branch may be out of
978 // range, so the machinery has to know about it.
David Goodwin5e47a9a2009-06-30 18:04:13 +0000979 int UncondBr = isThumb ? ((isThumb2) ? ARM::t2B : ARM::tB) : ARM::B;
Dale Johannesenb6728402009-02-13 02:25:56 +0000980 BuildMI(UserMBB, DebugLoc::getUnknownLoc(),
981 TII->get(UncondBr)).addMBB(*NewMBB);
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000982 unsigned MaxDisp = getUnconditionalBrDisp(UncondBr);
Bob Wilson84945262009-05-12 17:09:30 +0000983 ImmBranches.push_back(ImmBranch(&UserMBB->back(),
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000984 MaxDisp, false, UncondBr));
Evan Chengd3d9d662009-07-23 18:27:47 +0000985 int delta = isThumb1 ? 2 : 4;
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000986 BBSizes[UserMBB->getNumber()] += delta;
987 AdjustBBOffsetsAfter(UserMBB, delta);
988 } else {
989 // What a big block. Find a place within the block to split it.
Evan Chengd3d9d662009-07-23 18:27:47 +0000990 // This is a little tricky on Thumb1 since instructions are 2 bytes
Dale Johannesenb71aa2b2007-02-28 23:20:38 +0000991 // and constant pool entries are 4 bytes: if instruction I references
992 // island CPE, and instruction I+1 references CPE', it will
993 // not work well to put CPE as far forward as possible, since then
994 // CPE' cannot immediately follow it (that location is 2 bytes
995 // farther away from I+1 than CPE was from I) and we'd need to create
Dale Johannesen8593e412007-04-29 19:19:30 +0000996 // a new island. So, we make a first guess, then walk through the
997 // instructions between the one currently being looked at and the
998 // possible insertion point, and make sure any other instructions
999 // that reference CPEs will be able to use the same island area;
1000 // if not, we back up the insertion point.
1001
Dale Johannesenb71aa2b2007-02-28 23:20:38 +00001002 // The 4 in the following is for the unconditional branch we'll be
Evan Chengd3d9d662009-07-23 18:27:47 +00001003 // inserting (allows for long branch on Thumb1). Alignment of the
Dale Johannesen8593e412007-04-29 19:19:30 +00001004 // island is handled inside OffsetIsInRange.
1005 unsigned BaseInsertOffset = UserOffset + U.MaxDisp -4;
Dale Johannesenb71aa2b2007-02-28 23:20:38 +00001006 // This could point off the end of the block if we've already got
1007 // constant pool entries following this block; only the last one is
1008 // in the water list. Back past any possible branches (allow for a
1009 // conditional and a maximally long unconditional).
1010 if (BaseInsertOffset >= BBOffsets[UserMBB->getNumber()+1])
Bob Wilson84945262009-05-12 17:09:30 +00001011 BaseInsertOffset = BBOffsets[UserMBB->getNumber()+1] -
Evan Chengd3d9d662009-07-23 18:27:47 +00001012 (isThumb1 ? 6 : 8);
Dale Johannesenb71aa2b2007-02-28 23:20:38 +00001013 unsigned EndInsertOffset = BaseInsertOffset +
1014 CPEMI->getOperand(2).getImm();
1015 MachineBasicBlock::iterator MI = UserMI;
1016 ++MI;
1017 unsigned CPUIndex = CPUserIndex+1;
Nicolas Geoffray52e724a2008-04-16 20:10:13 +00001018 for (unsigned Offset = UserOffset+TII->GetInstSizeInBytes(UserMI);
Dale Johannesenb71aa2b2007-02-28 23:20:38 +00001019 Offset < BaseInsertOffset;
Nicolas Geoffray52e724a2008-04-16 20:10:13 +00001020 Offset += TII->GetInstSizeInBytes(MI),
Dale Johannesenb71aa2b2007-02-28 23:20:38 +00001021 MI = next(MI)) {
1022 if (CPUIndex < CPUsers.size() && CPUsers[CPUIndex].MI == MI) {
Evan Chengd3d9d662009-07-23 18:27:47 +00001023 CPUser &U = CPUsers[CPUIndex];
Bob Wilson84945262009-05-12 17:09:30 +00001024 if (!OffsetIsInRange(Offset, EndInsertOffset,
Evan Chengd3d9d662009-07-23 18:27:47 +00001025 U.MaxDisp, U.NegOk, U.IsSoImm)) {
1026 BaseInsertOffset -= (isThumb1 ? 2 : 4);
1027 EndInsertOffset -= (isThumb1 ? 2 : 4);
Dale Johannesenb71aa2b2007-02-28 23:20:38 +00001028 }
1029 // This is overly conservative, as we don't account for CPEMIs
1030 // being reused within the block, but it doesn't matter much.
1031 EndInsertOffset += CPUsers[CPUIndex].CPEMI->getOperand(2).getImm();
1032 CPUIndex++;
1033 }
1034 }
1035 DOUT << "Split in middle of big block\n";
1036 *NewMBB = SplitBlockBeforeInstr(prior(MI));
1037 }
1038}
1039
Dale Johannesen88e37ae2007-02-23 05:02:36 +00001040/// HandleConstantPoolUser - Analyze the specified user, checking to see if it
Bob Wilson39bf0512009-05-12 17:35:29 +00001041/// is out-of-range. If so, pick up the constant pool value and move it some
Dale Johannesen88e37ae2007-02-23 05:02:36 +00001042/// place in-range. Return true if we changed any addresses (thus must run
1043/// another pass of branch lengthening), false otherwise.
Bob Wilson84945262009-05-12 17:09:30 +00001044bool ARMConstantIslands::HandleConstantPoolUser(MachineFunction &Fn,
1045 unsigned CPUserIndex) {
Dale Johannesenf1b214d2007-02-28 18:41:23 +00001046 CPUser &U = CPUsers[CPUserIndex];
Dale Johannesen88e37ae2007-02-23 05:02:36 +00001047 MachineInstr *UserMI = U.MI;
1048 MachineInstr *CPEMI = U.CPEMI;
Chris Lattner8aa797a2007-12-30 23:10:15 +00001049 unsigned CPI = CPEMI->getOperand(1).getIndex();
Dale Johannesen88e37ae2007-02-23 05:02:36 +00001050 unsigned Size = CPEMI->getOperand(2).getImm();
Dale Johannesen88e37ae2007-02-23 05:02:36 +00001051 MachineBasicBlock *NewMBB;
Dale Johannesen8593e412007-04-29 19:19:30 +00001052 // Compute this only once, it's expensive. The 4 or 8 is the value the
Bob Wilson39bf0512009-05-12 17:35:29 +00001053 // hardware keeps in the PC (2 insns ahead of the reference).
Dale Johannesen88e37ae2007-02-23 05:02:36 +00001054 unsigned UserOffset = GetOffsetOf(UserMI) + (isThumb ? 4 : 8);
Evan Cheng768c9f72007-04-27 08:14:15 +00001055
Dale Johannesen88e37ae2007-02-23 05:02:36 +00001056 // See if the current entry is within range, or there is a clone of it
1057 // in range.
1058 int result = LookForExistingCPEntry(U, UserOffset);
1059 if (result==1) return false;
1060 else if (result==2) return true;
1061
1062 // No existing clone of this CPE is within range.
1063 // We will be generating a new clone. Get a UID for it.
Bob Wilson39bf0512009-05-12 17:35:29 +00001064 unsigned ID = AFI->createConstPoolEntryUId();
Dale Johannesen88e37ae2007-02-23 05:02:36 +00001065
1066 // Look for water where we can place this CPE. We look for the farthest one
1067 // away that will work. Forward references only for now (although later
1068 // we might find some that are backwards).
Dale Johannesen88e37ae2007-02-23 05:02:36 +00001069
Dale Johannesen8593e412007-04-29 19:19:30 +00001070 if (!LookForWater(U, UserOffset, &NewMBB)) {
Dale Johannesen88e37ae2007-02-23 05:02:36 +00001071 // No water found.
Dale Johannesen88e37ae2007-02-23 05:02:36 +00001072 DOUT << "No water found\n";
Dale Johannesenb71aa2b2007-02-28 23:20:38 +00001073 CreateNewWater(CPUserIndex, UserOffset, &NewMBB);
Dale Johannesen88e37ae2007-02-23 05:02:36 +00001074 }
1075
1076 // Okay, we know we can put an island before NewMBB now, do it!
Dan Gohman8e5f2c62008-07-07 23:14:23 +00001077 MachineBasicBlock *NewIsland = Fn.CreateMachineBasicBlock();
1078 Fn.insert(NewMBB, NewIsland);
Dale Johannesen88e37ae2007-02-23 05:02:36 +00001079
1080 // Update internal data structures to account for the newly inserted MBB.
1081 UpdateForInsertedWaterBlock(NewIsland);
1082
1083 // Decrement the old entry, and remove it if refcount becomes 0.
Evan Chenged884f32007-04-03 23:39:48 +00001084 DecrementOldEntry(CPI, CPEMI);
Dale Johannesen88e37ae2007-02-23 05:02:36 +00001085
1086 // Now that we have an island to add the CPE to, clone the original CPE and
1087 // add it to the island.
Dale Johannesenb6728402009-02-13 02:25:56 +00001088 U.CPEMI = BuildMI(NewIsland, DebugLoc::getUnknownLoc(),
1089 TII->get(ARM::CONSTPOOL_ENTRY))
Evan Chenga8e29892007-01-19 07:51:42 +00001090 .addImm(ID).addConstantPoolIndex(CPI).addImm(Size);
Dale Johannesen88e37ae2007-02-23 05:02:36 +00001091 CPEntries[CPI].push_back(CPEntry(U.CPEMI, ID, 1));
Evan Chengc99ef082007-02-09 20:54:44 +00001092 NumCPEs++;
1093
Dale Johannesen8593e412007-04-29 19:19:30 +00001094 BBOffsets[NewIsland->getNumber()] = BBOffsets[NewMBB->getNumber()];
Evan Chengb43216e2007-02-01 10:16:15 +00001095 // Compensate for .align 2 in thumb mode.
Bob Wilson84945262009-05-12 17:09:30 +00001096 if (isThumb && BBOffsets[NewIsland->getNumber()]%4 != 0)
Dale Johannesen8593e412007-04-29 19:19:30 +00001097 Size += 2;
Evan Chenga8e29892007-01-19 07:51:42 +00001098 // Increase the size of the island block to account for the new entry.
1099 BBSizes[NewIsland->getNumber()] += Size;
Dale Johannesen99c49a42007-02-25 00:47:03 +00001100 AdjustBBOffsetsAfter(NewIsland, Size);
Bob Wilson84945262009-05-12 17:09:30 +00001101
Evan Chenga8e29892007-01-19 07:51:42 +00001102 // Finally, change the CPI in the instruction operand to be ID.
1103 for (unsigned i = 0, e = UserMI->getNumOperands(); i != e; ++i)
Dan Gohmand735b802008-10-03 15:45:36 +00001104 if (UserMI->getOperand(i).isCPI()) {
Chris Lattner8aa797a2007-12-30 23:10:15 +00001105 UserMI->getOperand(i).setIndex(ID);
Evan Chenga8e29892007-01-19 07:51:42 +00001106 break;
1107 }
Bob Wilson84945262009-05-12 17:09:30 +00001108
Evan Chengc99ef082007-02-09 20:54:44 +00001109 DOUT << " Moved CPE to #" << ID << " CPI=" << CPI << "\t" << *UserMI;
Bob Wilson84945262009-05-12 17:09:30 +00001110
Evan Chenga8e29892007-01-19 07:51:42 +00001111 return true;
1112}
1113
Evan Chenged884f32007-04-03 23:39:48 +00001114/// RemoveDeadCPEMI - Remove a dead constant pool entry instruction. Update
1115/// sizes and offsets of impacted basic blocks.
1116void ARMConstantIslands::RemoveDeadCPEMI(MachineInstr *CPEMI) {
1117 MachineBasicBlock *CPEBB = CPEMI->getParent();
Dale Johannesen8593e412007-04-29 19:19:30 +00001118 unsigned Size = CPEMI->getOperand(2).getImm();
1119 CPEMI->eraseFromParent();
1120 BBSizes[CPEBB->getNumber()] -= Size;
1121 // All succeeding offsets have the current size value added in, fix this.
Evan Chenged884f32007-04-03 23:39:48 +00001122 if (CPEBB->empty()) {
Evan Chengd3d9d662009-07-23 18:27:47 +00001123 // In thumb1 mode, the size of island may be padded by two to compensate for
Dale Johannesen8593e412007-04-29 19:19:30 +00001124 // the alignment requirement. Then it will now be 2 when the block is
Evan Chenged884f32007-04-03 23:39:48 +00001125 // empty, so fix this.
1126 // All succeeding offsets have the current size value added in, fix this.
1127 if (BBSizes[CPEBB->getNumber()] != 0) {
Dale Johannesen8593e412007-04-29 19:19:30 +00001128 Size += BBSizes[CPEBB->getNumber()];
Evan Chenged884f32007-04-03 23:39:48 +00001129 BBSizes[CPEBB->getNumber()] = 0;
1130 }
Evan Chenged884f32007-04-03 23:39:48 +00001131 }
Dale Johannesen8593e412007-04-29 19:19:30 +00001132 AdjustBBOffsetsAfter(CPEBB, -Size);
1133 // An island has only one predecessor BB and one successor BB. Check if
1134 // this BB's predecessor jumps directly to this BB's successor. This
1135 // shouldn't happen currently.
1136 assert(!BBIsJumpedOver(CPEBB) && "How did this happen?");
1137 // FIXME: remove the empty blocks after all the work is done?
Evan Chenged884f32007-04-03 23:39:48 +00001138}
1139
1140/// RemoveUnusedCPEntries - Remove constant pool entries whose refcounts
1141/// are zero.
1142bool ARMConstantIslands::RemoveUnusedCPEntries() {
1143 unsigned MadeChange = false;
1144 for (unsigned i = 0, e = CPEntries.size(); i != e; ++i) {
1145 std::vector<CPEntry> &CPEs = CPEntries[i];
1146 for (unsigned j = 0, ee = CPEs.size(); j != ee; ++j) {
1147 if (CPEs[j].RefCount == 0 && CPEs[j].CPEMI) {
1148 RemoveDeadCPEMI(CPEs[j].CPEMI);
1149 CPEs[j].CPEMI = NULL;
1150 MadeChange = true;
1151 }
1152 }
Bob Wilson84945262009-05-12 17:09:30 +00001153 }
Evan Chenged884f32007-04-03 23:39:48 +00001154 return MadeChange;
1155}
1156
Dale Johannesen88e37ae2007-02-23 05:02:36 +00001157/// BBIsInRange - Returns true if the distance between specific MI and
Evan Cheng43aeab62007-01-26 20:38:26 +00001158/// specific BB can fit in MI's displacement field.
Evan Chengc0dbec72007-01-31 19:57:44 +00001159bool ARMConstantIslands::BBIsInRange(MachineInstr *MI,MachineBasicBlock *DestBB,
1160 unsigned MaxDisp) {
Dale Johannesenb71aa2b2007-02-28 23:20:38 +00001161 unsigned PCAdj = isThumb ? 4 : 8;
Evan Chengc0dbec72007-01-31 19:57:44 +00001162 unsigned BrOffset = GetOffsetOf(MI) + PCAdj;
Dale Johannesen99c49a42007-02-25 00:47:03 +00001163 unsigned DestOffset = BBOffsets[DestBB->getNumber()];
Evan Cheng43aeab62007-01-26 20:38:26 +00001164
Evan Chengc99ef082007-02-09 20:54:44 +00001165 DOUT << "Branch of destination BB#" << DestBB->getNumber()
1166 << " from BB#" << MI->getParent()->getNumber()
1167 << " max delta=" << MaxDisp
Dale Johannesen8593e412007-04-29 19:19:30 +00001168 << " from " << GetOffsetOf(MI) << " to " << DestOffset
1169 << " offset " << int(DestOffset-BrOffset) << "\t" << *MI;
Evan Chengc0dbec72007-01-31 19:57:44 +00001170
Dale Johannesen8593e412007-04-29 19:19:30 +00001171 if (BrOffset <= DestOffset) {
1172 // Branch before the Dest.
1173 if (DestOffset-BrOffset <= MaxDisp)
1174 return true;
1175 } else {
1176 if (BrOffset-DestOffset <= MaxDisp)
1177 return true;
1178 }
1179 return false;
Evan Cheng43aeab62007-01-26 20:38:26 +00001180}
1181
Evan Chengd1b2c1e2007-01-30 01:18:38 +00001182/// FixUpImmediateBr - Fix up an immediate branch whose destination is too far
1183/// away to fit in its displacement field.
1184bool ARMConstantIslands::FixUpImmediateBr(MachineFunction &Fn, ImmBranch &Br) {
Evan Chengaf5cbcb2007-01-25 03:12:46 +00001185 MachineInstr *MI = Br.MI;
Chris Lattner8aa797a2007-12-30 23:10:15 +00001186 MachineBasicBlock *DestBB = MI->getOperand(0).getMBB();
Evan Chengaf5cbcb2007-01-25 03:12:46 +00001187
Evan Chengc0dbec72007-01-31 19:57:44 +00001188 // Check to see if the DestBB is already in-range.
1189 if (BBIsInRange(MI, DestBB, Br.MaxDisp))
Evan Cheng43aeab62007-01-26 20:38:26 +00001190 return false;
Evan Chengaf5cbcb2007-01-25 03:12:46 +00001191
Evan Chengd1b2c1e2007-01-30 01:18:38 +00001192 if (!Br.isCond)
1193 return FixUpUnconditionalBr(Fn, Br);
1194 return FixUpConditionalBr(Fn, Br);
1195}
Evan Chengaf5cbcb2007-01-25 03:12:46 +00001196
Dale Johannesen88e37ae2007-02-23 05:02:36 +00001197/// FixUpUnconditionalBr - Fix up an unconditional branch whose destination is
1198/// too far away to fit in its displacement field. If the LR register has been
Evan Chengd1b2c1e2007-01-30 01:18:38 +00001199/// spilled in the epilogue, then we can use BL to implement a far jump.
Bob Wilson39bf0512009-05-12 17:35:29 +00001200/// Otherwise, add an intermediate branch instruction to a branch.
Evan Chengd1b2c1e2007-01-30 01:18:38 +00001201bool
1202ARMConstantIslands::FixUpUnconditionalBr(MachineFunction &Fn, ImmBranch &Br) {
1203 MachineInstr *MI = Br.MI;
1204 MachineBasicBlock *MBB = MI->getParent();
Evan Chengd3d9d662009-07-23 18:27:47 +00001205 assert(isThumb && !isThumb2 && "Expected a Thumb1 function!");
Evan Chengd1b2c1e2007-01-30 01:18:38 +00001206
1207 // Use BL to implement far jump.
1208 Br.MaxDisp = (1 << 21) * 2;
Chris Lattner5080f4d2008-01-11 18:10:50 +00001209 MI->setDesc(TII->get(ARM::tBfar));
Evan Chengd1b2c1e2007-01-30 01:18:38 +00001210 BBSizes[MBB->getNumber()] += 2;
Dale Johannesen99c49a42007-02-25 00:47:03 +00001211 AdjustBBOffsetsAfter(MBB, 2);
Evan Chengd1b2c1e2007-01-30 01:18:38 +00001212 HasFarJump = true;
1213 NumUBrFixed++;
Evan Chengbd5d3db2007-02-03 02:08:34 +00001214
Evan Chengc99ef082007-02-09 20:54:44 +00001215 DOUT << " Changed B to long jump " << *MI;
Evan Chengbd5d3db2007-02-03 02:08:34 +00001216
Evan Chengd1b2c1e2007-01-30 01:18:38 +00001217 return true;
1218}
1219
Dale Johannesen88e37ae2007-02-23 05:02:36 +00001220/// FixUpConditionalBr - Fix up a conditional branch whose destination is too
Evan Chengd1b2c1e2007-01-30 01:18:38 +00001221/// far away to fit in its displacement field. It is converted to an inverse
1222/// conditional branch + an unconditional branch to the destination.
1223bool
1224ARMConstantIslands::FixUpConditionalBr(MachineFunction &Fn, ImmBranch &Br) {
1225 MachineInstr *MI = Br.MI;
Chris Lattner8aa797a2007-12-30 23:10:15 +00001226 MachineBasicBlock *DestBB = MI->getOperand(0).getMBB();
Evan Chengd1b2c1e2007-01-30 01:18:38 +00001227
Bob Wilson39bf0512009-05-12 17:35:29 +00001228 // Add an unconditional branch to the destination and invert the branch
Evan Chengd1b2c1e2007-01-30 01:18:38 +00001229 // condition to jump over it:
Evan Chengaf5cbcb2007-01-25 03:12:46 +00001230 // blt L1
1231 // =>
1232 // bge L2
1233 // b L1
1234 // L2:
Chris Lattner9a1ceae2007-12-30 20:49:49 +00001235 ARMCC::CondCodes CC = (ARMCC::CondCodes)MI->getOperand(1).getImm();
Evan Chengaf5cbcb2007-01-25 03:12:46 +00001236 CC = ARMCC::getOppositeCondition(CC);
Evan Cheng0e1d3792007-07-05 07:18:20 +00001237 unsigned CCReg = MI->getOperand(2).getReg();
Evan Chengaf5cbcb2007-01-25 03:12:46 +00001238
1239 // If the branch is at the end of its MBB and that has a fall-through block,
1240 // direct the updated conditional branch to the fall-through block. Otherwise,
1241 // split the MBB before the next instruction.
1242 MachineBasicBlock *MBB = MI->getParent();
Evan Chengbd5d3db2007-02-03 02:08:34 +00001243 MachineInstr *BMI = &MBB->back();
1244 bool NeedSplit = (BMI != MI) || !BBHasFallthrough(MBB);
Evan Cheng43aeab62007-01-26 20:38:26 +00001245
Evan Chengd1b2c1e2007-01-30 01:18:38 +00001246 NumCBrFixed++;
Evan Chengbd5d3db2007-02-03 02:08:34 +00001247 if (BMI != MI) {
Dan Gohman8e5f2c62008-07-07 23:14:23 +00001248 if (next(MachineBasicBlock::iterator(MI)) == prior(MBB->end()) &&
Evan Chengbd5d3db2007-02-03 02:08:34 +00001249 BMI->getOpcode() == Br.UncondBr) {
Bob Wilson39bf0512009-05-12 17:35:29 +00001250 // Last MI in the BB is an unconditional branch. Can we simply invert the
Evan Cheng43aeab62007-01-26 20:38:26 +00001251 // condition and swap destinations:
1252 // beq L1
1253 // b L2
1254 // =>
1255 // bne L2
1256 // b L1
Chris Lattner8aa797a2007-12-30 23:10:15 +00001257 MachineBasicBlock *NewDest = BMI->getOperand(0).getMBB();
Evan Chengc0dbec72007-01-31 19:57:44 +00001258 if (BBIsInRange(MI, NewDest, Br.MaxDisp)) {
Evan Chengc99ef082007-02-09 20:54:44 +00001259 DOUT << " Invert Bcc condition and swap its destination with " << *BMI;
Chris Lattner8aa797a2007-12-30 23:10:15 +00001260 BMI->getOperand(0).setMBB(DestBB);
1261 MI->getOperand(0).setMBB(NewDest);
Evan Cheng43aeab62007-01-26 20:38:26 +00001262 MI->getOperand(1).setImm(CC);
1263 return true;
1264 }
1265 }
1266 }
1267
1268 if (NeedSplit) {
Evan Chengaf5cbcb2007-01-25 03:12:46 +00001269 SplitBlockBeforeInstr(MI);
Bob Wilson39bf0512009-05-12 17:35:29 +00001270 // No need for the branch to the next block. We're adding an unconditional
Evan Chengdd353b82007-01-26 02:02:39 +00001271 // branch to the destination.
Nicolas Geoffray52e724a2008-04-16 20:10:13 +00001272 int delta = TII->GetInstSizeInBytes(&MBB->back());
Dale Johannesen56c42ef2007-04-23 20:09:04 +00001273 BBSizes[MBB->getNumber()] -= delta;
Dale Johannesen8593e412007-04-29 19:19:30 +00001274 MachineBasicBlock* SplitBB = next(MachineFunction::iterator(MBB));
1275 AdjustBBOffsetsAfter(SplitBB, -delta);
Evan Chengdd353b82007-01-26 02:02:39 +00001276 MBB->back().eraseFromParent();
Dale Johannesen8593e412007-04-29 19:19:30 +00001277 // BBOffsets[SplitBB] is wrong temporarily, fixed below
Evan Chengdd353b82007-01-26 02:02:39 +00001278 }
Evan Chengaf5cbcb2007-01-25 03:12:46 +00001279 MachineBasicBlock *NextBB = next(MachineFunction::iterator(MBB));
Bob Wilson84945262009-05-12 17:09:30 +00001280
Evan Chengc99ef082007-02-09 20:54:44 +00001281 DOUT << " Insert B to BB#" << DestBB->getNumber()
1282 << " also invert condition and change dest. to BB#"
1283 << NextBB->getNumber() << "\n";
Evan Chengaf5cbcb2007-01-25 03:12:46 +00001284
Dale Johannesen56c42ef2007-04-23 20:09:04 +00001285 // Insert a new conditional branch and a new unconditional branch.
Evan Chengaf5cbcb2007-01-25 03:12:46 +00001286 // Also update the ImmBranch as well as adding a new entry for the new branch.
Dale Johannesenb6728402009-02-13 02:25:56 +00001287 BuildMI(MBB, DebugLoc::getUnknownLoc(),
1288 TII->get(MI->getOpcode()))
1289 .addMBB(NextBB).addImm(CC).addReg(CCReg);
Evan Chengaf5cbcb2007-01-25 03:12:46 +00001290 Br.MI = &MBB->back();
Nicolas Geoffray52e724a2008-04-16 20:10:13 +00001291 BBSizes[MBB->getNumber()] += TII->GetInstSizeInBytes(&MBB->back());
Dale Johannesenb6728402009-02-13 02:25:56 +00001292 BuildMI(MBB, DebugLoc::getUnknownLoc(), TII->get(Br.UncondBr)).addMBB(DestBB);
Nicolas Geoffray52e724a2008-04-16 20:10:13 +00001293 BBSizes[MBB->getNumber()] += TII->GetInstSizeInBytes(&MBB->back());
Evan Chenga9b8b8d2007-01-31 18:29:27 +00001294 unsigned MaxDisp = getUnconditionalBrDisp(Br.UncondBr);
Evan Chenga0bf7942007-01-25 23:31:04 +00001295 ImmBranches.push_back(ImmBranch(&MBB->back(), MaxDisp, false, Br.UncondBr));
Dale Johannesen56c42ef2007-04-23 20:09:04 +00001296
1297 // Remove the old conditional branch. It may or may not still be in MBB.
Nicolas Geoffray52e724a2008-04-16 20:10:13 +00001298 BBSizes[MI->getParent()->getNumber()] -= TII->GetInstSizeInBytes(MI);
Evan Chengaf5cbcb2007-01-25 03:12:46 +00001299 MI->eraseFromParent();
1300
Dale Johannesen56c42ef2007-04-23 20:09:04 +00001301 // The net size change is an addition of one unconditional branch.
Nicolas Geoffray52e724a2008-04-16 20:10:13 +00001302 int delta = TII->GetInstSizeInBytes(&MBB->back());
Dale Johannesen99c49a42007-02-25 00:47:03 +00001303 AdjustBBOffsetsAfter(MBB, delta);
Evan Chengaf5cbcb2007-01-25 03:12:46 +00001304 return true;
1305}
Evan Chengd1b2c1e2007-01-30 01:18:38 +00001306
Evan Chengd1b2c1e2007-01-30 01:18:38 +00001307/// UndoLRSpillRestore - Remove Thumb push / pop instructions that only spills
1308/// LR / restores LR to pc.
1309bool ARMConstantIslands::UndoLRSpillRestore() {
1310 bool MadeChange = false;
1311 for (unsigned i = 0, e = PushPopMIs.size(); i != e; ++i) {
1312 MachineInstr *MI = PushPopMIs[i];
Evan Cheng44bec522007-05-15 01:29:07 +00001313 if (MI->getOpcode() == ARM::tPOP_RET &&
1314 MI->getOperand(0).getReg() == ARM::PC &&
1315 MI->getNumExplicitOperands() == 1) {
Dale Johannesenb6728402009-02-13 02:25:56 +00001316 BuildMI(MI->getParent(), MI->getDebugLoc(), TII->get(ARM::tBX_RET));
Evan Cheng44bec522007-05-15 01:29:07 +00001317 MI->eraseFromParent();
1318 MadeChange = true;
Evan Chengd1b2c1e2007-01-30 01:18:38 +00001319 }
1320 }
1321 return MadeChange;
1322}