blob: 9cceb4e8d745b29b45954f56437855855e1c0789 [file] [log] [blame]
Dan Gohmana629b482008-12-08 17:50:35 +00001//===---- ScheduleDAGInstrs.cpp - MachineInstr Rescheduling ---------------===//
Dan Gohman343f0c02008-11-19 23:18:57 +00002//
3// The LLVM Compiler Infrastructure
4//
5// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
9//
Dan Gohmana629b482008-12-08 17:50:35 +000010// This implements the ScheduleDAGInstrs class, which implements re-scheduling
11// of MachineInstrs.
Dan Gohman343f0c02008-11-19 23:18:57 +000012//
13//===----------------------------------------------------------------------===//
14
15#define DEBUG_TYPE "sched-instrs"
Dan Gohman6dc75fe2009-02-06 17:12:10 +000016#include "ScheduleDAGInstrs.h"
Dan Gohman8906f952009-07-17 20:58:59 +000017#include "llvm/Operator.h"
Dan Gohman3311a1f2009-01-30 02:49:14 +000018#include "llvm/Analysis/AliasAnalysis.h"
Dan Gohman5034dd32010-12-15 20:02:24 +000019#include "llvm/Analysis/ValueTracking.h"
Dan Gohman3f237442008-12-16 03:25:46 +000020#include "llvm/CodeGen/MachineFunctionPass.h"
Dan Gohmanc76909a2009-09-25 20:36:54 +000021#include "llvm/CodeGen/MachineMemOperand.h"
Dan Gohman3f237442008-12-16 03:25:46 +000022#include "llvm/CodeGen/MachineRegisterInfo.h"
Dan Gohman6a9041e2008-12-04 01:35:46 +000023#include "llvm/CodeGen/PseudoSourceValue.h"
Evan Chengab8be962011-06-29 01:14:12 +000024#include "llvm/MC/MCInstrItineraries.h"
Dan Gohman343f0c02008-11-19 23:18:57 +000025#include "llvm/Target/TargetMachine.h"
26#include "llvm/Target/TargetInstrInfo.h"
27#include "llvm/Target/TargetRegisterInfo.h"
Dan Gohman3f237442008-12-16 03:25:46 +000028#include "llvm/Target/TargetSubtarget.h"
Dan Gohman343f0c02008-11-19 23:18:57 +000029#include "llvm/Support/Debug.h"
30#include "llvm/Support/raw_ostream.h"
Dan Gohman3f237442008-12-16 03:25:46 +000031#include "llvm/ADT/SmallSet.h"
Dan Gohman343f0c02008-11-19 23:18:57 +000032using namespace llvm;
33
Dan Gohman79ce2762009-01-15 19:20:50 +000034ScheduleDAGInstrs::ScheduleDAGInstrs(MachineFunction &mf,
Dan Gohman3f237442008-12-16 03:25:46 +000035 const MachineLoopInfo &mli,
36 const MachineDominatorTree &mdt)
Evan Cheng3ef1c872010-09-10 01:29:16 +000037 : ScheduleDAG(mf), MLI(mli), MDT(mdt), MFI(mf.getFrameInfo()),
38 InstrItins(mf.getTarget().getInstrItineraryData()),
Devang Patelcf4cc842011-06-02 20:07:12 +000039 Defs(TRI->getNumRegs()), Uses(TRI->getNumRegs()),
Devang Patele29e8e12011-06-02 21:26:52 +000040 LoopRegs(MLI, MDT), FirstDbgValue(0) {
Devang Patelcf4cc842011-06-02 20:07:12 +000041 DbgValues.clear();
Evan Cheng38bdfc62009-10-18 19:58:47 +000042}
Dan Gohman343f0c02008-11-19 23:18:57 +000043
Dan Gohman47ac0f02009-02-11 04:27:20 +000044/// Run - perform scheduling.
45///
46void ScheduleDAGInstrs::Run(MachineBasicBlock *bb,
47 MachineBasicBlock::iterator begin,
48 MachineBasicBlock::iterator end,
49 unsigned endcount) {
50 BB = bb;
51 Begin = begin;
52 InsertPosIndex = endcount;
53
54 ScheduleDAG::Run(bb, end);
55}
56
Dan Gohman3311a1f2009-01-30 02:49:14 +000057/// getUnderlyingObjectFromInt - This is the function that does the work of
58/// looking through basic ptrtoint+arithmetic+inttoptr sequences.
59static const Value *getUnderlyingObjectFromInt(const Value *V) {
60 do {
Dan Gohman8906f952009-07-17 20:58:59 +000061 if (const Operator *U = dyn_cast<Operator>(V)) {
Dan Gohman3311a1f2009-01-30 02:49:14 +000062 // If we find a ptrtoint, we can transfer control back to the
63 // regular getUnderlyingObjectFromInt.
Dan Gohman8906f952009-07-17 20:58:59 +000064 if (U->getOpcode() == Instruction::PtrToInt)
Dan Gohman3311a1f2009-01-30 02:49:14 +000065 return U->getOperand(0);
66 // If we find an add of a constant or a multiplied value, it's
67 // likely that the other operand will lead us to the base
68 // object. We don't have to worry about the case where the
Dan Gohman748f98f2009-08-07 01:26:06 +000069 // object address is somehow being computed by the multiply,
Dan Gohman3311a1f2009-01-30 02:49:14 +000070 // because our callers only care when the result is an
71 // identifibale object.
Dan Gohman8906f952009-07-17 20:58:59 +000072 if (U->getOpcode() != Instruction::Add ||
Dan Gohman3311a1f2009-01-30 02:49:14 +000073 (!isa<ConstantInt>(U->getOperand(1)) &&
Dan Gohman8906f952009-07-17 20:58:59 +000074 Operator::getOpcode(U->getOperand(1)) != Instruction::Mul))
Dan Gohman3311a1f2009-01-30 02:49:14 +000075 return V;
76 V = U->getOperand(0);
77 } else {
78 return V;
79 }
Duncan Sands1df98592010-02-16 11:11:14 +000080 assert(V->getType()->isIntegerTy() && "Unexpected operand type!");
Dan Gohman3311a1f2009-01-30 02:49:14 +000081 } while (1);
82}
83
Dan Gohman5034dd32010-12-15 20:02:24 +000084/// getUnderlyingObject - This is a wrapper around GetUnderlyingObject
Dan Gohman3311a1f2009-01-30 02:49:14 +000085/// and adds support for basic ptrtoint+arithmetic+inttoptr sequences.
86static const Value *getUnderlyingObject(const Value *V) {
87 // First just call Value::getUnderlyingObject to let it do what it does.
88 do {
Dan Gohman5034dd32010-12-15 20:02:24 +000089 V = GetUnderlyingObject(V);
Dan Gohman3311a1f2009-01-30 02:49:14 +000090 // If it found an inttoptr, use special code to continue climing.
Dan Gohman8906f952009-07-17 20:58:59 +000091 if (Operator::getOpcode(V) != Instruction::IntToPtr)
Dan Gohman3311a1f2009-01-30 02:49:14 +000092 break;
93 const Value *O = getUnderlyingObjectFromInt(cast<User>(V)->getOperand(0));
94 // If that succeeded in finding a pointer, continue the search.
Duncan Sands1df98592010-02-16 11:11:14 +000095 if (!O->getType()->isPointerTy())
Dan Gohman3311a1f2009-01-30 02:49:14 +000096 break;
97 V = O;
98 } while (1);
99 return V;
100}
101
102/// getUnderlyingObjectForInstr - If this machine instr has memory reference
103/// information and it can be tracked to a normal reference to a known
104/// object, return the Value for that object. Otherwise return null.
Evan Cheng38bdfc62009-10-18 19:58:47 +0000105static const Value *getUnderlyingObjectForInstr(const MachineInstr *MI,
David Goodwina9e61072009-11-03 20:15:00 +0000106 const MachineFrameInfo *MFI,
107 bool &MayAlias) {
108 MayAlias = true;
Dan Gohman3311a1f2009-01-30 02:49:14 +0000109 if (!MI->hasOneMemOperand() ||
Dan Gohmanc76909a2009-09-25 20:36:54 +0000110 !(*MI->memoperands_begin())->getValue() ||
111 (*MI->memoperands_begin())->isVolatile())
Dan Gohman3311a1f2009-01-30 02:49:14 +0000112 return 0;
113
Dan Gohmanc76909a2009-09-25 20:36:54 +0000114 const Value *V = (*MI->memoperands_begin())->getValue();
Dan Gohman3311a1f2009-01-30 02:49:14 +0000115 if (!V)
116 return 0;
117
118 V = getUnderlyingObject(V);
Evan Chengff89dcb2009-10-18 18:16:27 +0000119 if (const PseudoSourceValue *PSV = dyn_cast<PseudoSourceValue>(V)) {
120 // For now, ignore PseudoSourceValues which may alias LLVM IR values
121 // because the code that uses this function has no way to cope with
122 // such aliases.
Evan Cheng38bdfc62009-10-18 19:58:47 +0000123 if (PSV->isAliased(MFI))
Evan Chengff89dcb2009-10-18 18:16:27 +0000124 return 0;
Andrew Trickf405b1a2011-05-05 19:24:06 +0000125
David Goodwin980d4942009-11-09 19:22:17 +0000126 MayAlias = PSV->mayAlias(MFI);
Evan Chengff89dcb2009-10-18 18:16:27 +0000127 return V;
128 }
Dan Gohman3311a1f2009-01-30 02:49:14 +0000129
Evan Chengff89dcb2009-10-18 18:16:27 +0000130 if (isIdentifiedObject(V))
131 return V;
132
133 return 0;
Dan Gohman3311a1f2009-01-30 02:49:14 +0000134}
135
Dan Gohman9e64bbb2009-02-10 23:27:53 +0000136void ScheduleDAGInstrs::StartBlock(MachineBasicBlock *BB) {
137 if (MachineLoop *ML = MLI.getLoopFor(BB))
138 if (BB == ML->getLoopLatch()) {
139 MachineBasicBlock *Header = ML->getHeader();
140 for (MachineBasicBlock::livein_iterator I = Header->livein_begin(),
141 E = Header->livein_end(); I != E; ++I)
142 LoopLiveInRegs.insert(*I);
143 LoopRegs.VisitLoop(ML);
144 }
145}
146
Evan Chengec6906b2010-10-23 02:10:46 +0000147/// AddSchedBarrierDeps - Add dependencies from instructions in the current
148/// list of instructions being scheduled to scheduling barrier by adding
149/// the exit SU to the register defs and use list. This is because we want to
150/// make sure instructions which define registers that are either used by
151/// the terminator or are live-out are properly scheduled. This is
152/// especially important when the definition latency of the return value(s)
153/// are too high to be hidden by the branch or when the liveout registers
154/// used by instructions in the fallthrough block.
155void ScheduleDAGInstrs::AddSchedBarrierDeps() {
156 MachineInstr *ExitMI = InsertPos != BB->end() ? &*InsertPos : 0;
157 ExitSU.setInstr(ExitMI);
158 bool AllDepKnown = ExitMI &&
159 (ExitMI->getDesc().isCall() || ExitMI->getDesc().isBarrier());
160 if (ExitMI && AllDepKnown) {
161 // If it's a call or a barrier, add dependencies on the defs and uses of
162 // instruction.
163 for (unsigned i = 0, e = ExitMI->getNumOperands(); i != e; ++i) {
164 const MachineOperand &MO = ExitMI->getOperand(i);
165 if (!MO.isReg() || MO.isDef()) continue;
166 unsigned Reg = MO.getReg();
167 if (Reg == 0) continue;
168
169 assert(TRI->isPhysicalRegister(Reg) && "Virtual register encountered!");
170 Uses[Reg].push_back(&ExitSU);
171 }
172 } else {
173 // For others, e.g. fallthrough, conditional branch, assume the exit
Evan Chengde5fa932010-10-27 23:17:17 +0000174 // uses all the registers that are livein to the successor blocks.
175 SmallSet<unsigned, 8> Seen;
176 for (MachineBasicBlock::succ_iterator SI = BB->succ_begin(),
177 SE = BB->succ_end(); SI != SE; ++SI)
178 for (MachineBasicBlock::livein_iterator I = (*SI)->livein_begin(),
Andrew Trickf405b1a2011-05-05 19:24:06 +0000179 E = (*SI)->livein_end(); I != E; ++I) {
Evan Chengde5fa932010-10-27 23:17:17 +0000180 unsigned Reg = *I;
181 if (Seen.insert(Reg))
182 Uses[Reg].push_back(&ExitSU);
183 }
Evan Chengec6906b2010-10-23 02:10:46 +0000184 }
185}
186
Dan Gohmana70dca12009-10-09 23:27:56 +0000187void ScheduleDAGInstrs::BuildSchedGraph(AliasAnalysis *AA) {
Dan Gohman9e64bbb2009-02-10 23:27:53 +0000188 // We'll be allocating one SUnit for each instruction, plus one for
189 // the region exit node.
Dan Gohman343f0c02008-11-19 23:18:57 +0000190 SUnits.reserve(BB->size());
191
Dan Gohman6a9041e2008-12-04 01:35:46 +0000192 // We build scheduling units by walking a block's instruction list from bottom
193 // to top.
194
David Goodwin980d4942009-11-09 19:22:17 +0000195 // Remember where a generic side-effecting instruction is as we procede.
196 SUnit *BarrierChain = 0, *AliasChain = 0;
Dan Gohman6a9041e2008-12-04 01:35:46 +0000197
David Goodwin980d4942009-11-09 19:22:17 +0000198 // Memory references to specific known memory locations are tracked
199 // so that they can be given more precise dependencies. We track
200 // separately the known memory locations that may alias and those
201 // that are known not to alias
202 std::map<const Value *, SUnit *> AliasMemDefs, NonAliasMemDefs;
203 std::map<const Value *, std::vector<SUnit *> > AliasMemUses, NonAliasMemUses;
Dan Gohman6a9041e2008-12-04 01:35:46 +0000204
Dan Gohman3f237442008-12-16 03:25:46 +0000205 // Check to see if the scheduler cares about latencies.
206 bool UnitLatencies = ForceUnitLatencies();
207
Dan Gohman8749b612008-12-16 03:35:01 +0000208 // Ask the target if address-backscheduling is desirable, and if so how much.
David Goodwin71046162009-08-13 16:05:04 +0000209 const TargetSubtarget &ST = TM.getSubtarget<TargetSubtarget>();
210 unsigned SpecialAddressLatency = ST.getSpecialAddressLatency();
Dan Gohman8749b612008-12-16 03:35:01 +0000211
Dale Johannesenbfdf7f32010-03-10 22:13:47 +0000212 // Remove any stale debug info; sometimes BuildSchedGraph is called again
213 // without emitting the info from the previous call.
Devang Patelcf4cc842011-06-02 20:07:12 +0000214 DbgValues.clear();
215 FirstDbgValue = NULL;
Dale Johannesenbfdf7f32010-03-10 22:13:47 +0000216
Evan Chengec6906b2010-10-23 02:10:46 +0000217 // Model data dependencies between instructions being scheduled and the
218 // ExitSU.
219 AddSchedBarrierDeps();
220
Andrew Trick9b668532011-05-06 21:52:52 +0000221 for (int i = 0, e = TRI->getNumRegs(); i != e; ++i) {
222 assert(Defs[i].empty() && "Only BuildGraph should push/pop Defs");
223 }
224
Dan Gohman9e64bbb2009-02-10 23:27:53 +0000225 // Walk the list of instructions, from bottom moving up.
Devang Patelcf4cc842011-06-02 20:07:12 +0000226 MachineInstr *PrevMI = NULL;
Dan Gohman47ac0f02009-02-11 04:27:20 +0000227 for (MachineBasicBlock::iterator MII = InsertPos, MIE = Begin;
Dan Gohman343f0c02008-11-19 23:18:57 +0000228 MII != MIE; --MII) {
229 MachineInstr *MI = prior(MII);
Devang Patelcf4cc842011-06-02 20:07:12 +0000230 if (MI && PrevMI) {
231 DbgValues.push_back(std::make_pair(PrevMI, MI));
232 PrevMI = NULL;
233 }
234
Dale Johannesenbfdf7f32010-03-10 22:13:47 +0000235 if (MI->isDebugValue()) {
Devang Patelcf4cc842011-06-02 20:07:12 +0000236 PrevMI = MI;
Dale Johannesenbfdf7f32010-03-10 22:13:47 +0000237 continue;
238 }
Devang Patelcf4cc842011-06-02 20:07:12 +0000239
Evan Chenge837dea2011-06-28 19:10:37 +0000240 const MCInstrDesc &MCID = MI->getDesc();
241 assert(!MCID.isTerminator() && !MI->isLabel() &&
Dan Gohman9e64bbb2009-02-10 23:27:53 +0000242 "Cannot schedule terminators or labels!");
243 // Create the SUnit for this MI.
Dan Gohman343f0c02008-11-19 23:18:57 +0000244 SUnit *SU = NewSUnit(MI);
Evan Chenge837dea2011-06-28 19:10:37 +0000245 SU->isCall = MCID.isCall();
246 SU->isCommutable = MCID.isCommutable();
Dan Gohman343f0c02008-11-19 23:18:57 +0000247
Dan Gohman54e4c362008-12-09 22:54:47 +0000248 // Assign the Latency field of SU using target-provided information.
Dan Gohman3f237442008-12-16 03:25:46 +0000249 if (UnitLatencies)
250 SU->Latency = 1;
251 else
252 ComputeLatency(SU);
Dan Gohman54e4c362008-12-09 22:54:47 +0000253
Dan Gohman6a9041e2008-12-04 01:35:46 +0000254 // Add register-based dependencies (data, anti, and output).
Dan Gohman343f0c02008-11-19 23:18:57 +0000255 for (unsigned j = 0, n = MI->getNumOperands(); j != n; ++j) {
256 const MachineOperand &MO = MI->getOperand(j);
257 if (!MO.isReg()) continue;
258 unsigned Reg = MO.getReg();
259 if (Reg == 0) continue;
260
261 assert(TRI->isPhysicalRegister(Reg) && "Virtual register encountered!");
Dale Johannesenbfdf7f32010-03-10 22:13:47 +0000262
Dan Gohman343f0c02008-11-19 23:18:57 +0000263 std::vector<SUnit *> &UseList = Uses[Reg];
Andrew Trick9b668532011-05-06 21:52:52 +0000264 // Defs are push in the order they are visited and never reordered.
Dan Gohman3f237442008-12-16 03:25:46 +0000265 std::vector<SUnit *> &DefList = Defs[Reg];
David Goodwind94a4e52009-08-10 15:55:25 +0000266 // Optionally add output and anti dependencies. For anti
267 // dependencies we use a latency of 0 because for a multi-issue
268 // target we want to allow the defining instruction to issue
269 // in the same cycle as the using instruction.
270 // TODO: Using a latency of 1 here for output dependencies assumes
271 // there's no cost for reusing registers.
Dan Gohman54e4c362008-12-09 22:54:47 +0000272 SDep::Kind Kind = MO.isUse() ? SDep::Anti : SDep::Output;
David Goodwind94a4e52009-08-10 15:55:25 +0000273 unsigned AOLatency = (Kind == SDep::Anti) ? 0 : 1;
Dan Gohman3f237442008-12-16 03:25:46 +0000274 for (unsigned i = 0, e = DefList.size(); i != e; ++i) {
275 SUnit *DefSU = DefList[i];
Evan Chengec6906b2010-10-23 02:10:46 +0000276 if (DefSU == &ExitSU)
277 continue;
Dan Gohman3f237442008-12-16 03:25:46 +0000278 if (DefSU != SU &&
279 (Kind != SDep::Output || !MO.isDead() ||
280 !DefSU->getInstr()->registerDefIsDead(Reg)))
David Goodwind94a4e52009-08-10 15:55:25 +0000281 DefSU->addPred(SDep(SU, Kind, AOLatency, /*Reg=*/Reg));
Dan Gohman3f237442008-12-16 03:25:46 +0000282 }
Dan Gohman343f0c02008-11-19 23:18:57 +0000283 for (const unsigned *Alias = TRI->getAliasSet(Reg); *Alias; ++Alias) {
Andrew Trick9b668532011-05-06 21:52:52 +0000284 std::vector<SUnit *> &MemDefList = Defs[*Alias];
285 for (unsigned i = 0, e = MemDefList.size(); i != e; ++i) {
286 SUnit *DefSU = MemDefList[i];
Evan Chengec6906b2010-10-23 02:10:46 +0000287 if (DefSU == &ExitSU)
288 continue;
Dan Gohman3f237442008-12-16 03:25:46 +0000289 if (DefSU != SU &&
290 (Kind != SDep::Output || !MO.isDead() ||
Dan Gohman91203cf2009-10-26 18:26:18 +0000291 !DefSU->getInstr()->registerDefIsDead(*Alias)))
David Goodwind94a4e52009-08-10 15:55:25 +0000292 DefSU->addPred(SDep(SU, Kind, AOLatency, /*Reg=*/ *Alias));
Dan Gohman3f237442008-12-16 03:25:46 +0000293 }
Dan Gohman343f0c02008-11-19 23:18:57 +0000294 }
295
296 if (MO.isDef()) {
297 // Add any data dependencies.
Dan Gohman3f237442008-12-16 03:25:46 +0000298 unsigned DataLatency = SU->Latency;
299 for (unsigned i = 0, e = UseList.size(); i != e; ++i) {
300 SUnit *UseSU = UseList[i];
Evan Chenga69ec092010-03-22 21:24:33 +0000301 if (UseSU == SU)
302 continue;
303 unsigned LDataLatency = DataLatency;
304 // Optionally add in a special extra latency for nodes that
305 // feed addresses.
306 // TODO: Do this for register aliases too.
307 // TODO: Perhaps we should get rid of
308 // SpecialAddressLatency and just move this into
309 // adjustSchedDependency for the targets that care about it.
Evan Chengec6906b2010-10-23 02:10:46 +0000310 if (SpecialAddressLatency != 0 && !UnitLatencies &&
311 UseSU != &ExitSU) {
Evan Chenga69ec092010-03-22 21:24:33 +0000312 MachineInstr *UseMI = UseSU->getInstr();
Evan Chenge837dea2011-06-28 19:10:37 +0000313 const MCInstrDesc &UseMCID = UseMI->getDesc();
Evan Chenga69ec092010-03-22 21:24:33 +0000314 int RegUseIndex = UseMI->findRegisterUseOperandIdx(Reg);
315 assert(RegUseIndex >= 0 && "UseMI doesn's use register!");
Evan Chengec6906b2010-10-23 02:10:46 +0000316 if (RegUseIndex >= 0 &&
Evan Chenge837dea2011-06-28 19:10:37 +0000317 (UseMCID.mayLoad() || UseMCID.mayStore()) &&
318 (unsigned)RegUseIndex < UseMCID.getNumOperands() &&
319 UseMCID.OpInfo[RegUseIndex].isLookupPtrRegClass())
Evan Chenga69ec092010-03-22 21:24:33 +0000320 LDataLatency += SpecialAddressLatency;
Dan Gohman3f237442008-12-16 03:25:46 +0000321 }
Evan Chenga69ec092010-03-22 21:24:33 +0000322 // Adjust the dependence latency using operand def/use
323 // information (if any), and then allow the target to
324 // perform its own adjustments.
325 const SDep& dep = SDep(SU, SDep::Data, LDataLatency, Reg);
326 if (!UnitLatencies) {
Dan Gohman3fb150a2010-04-17 17:42:52 +0000327 ComputeOperandLatency(SU, UseSU, const_cast<SDep &>(dep));
328 ST.adjustSchedDependency(SU, UseSU, const_cast<SDep &>(dep));
Evan Chenga69ec092010-03-22 21:24:33 +0000329 }
330 UseSU->addPred(dep);
Dan Gohman3f237442008-12-16 03:25:46 +0000331 }
Dan Gohman343f0c02008-11-19 23:18:57 +0000332 for (const unsigned *Alias = TRI->getAliasSet(Reg); *Alias; ++Alias) {
333 std::vector<SUnit *> &UseList = Uses[*Alias];
Dan Gohman3f237442008-12-16 03:25:46 +0000334 for (unsigned i = 0, e = UseList.size(); i != e; ++i) {
335 SUnit *UseSU = UseList[i];
Evan Chenga69ec092010-03-22 21:24:33 +0000336 if (UseSU == SU)
337 continue;
338 const SDep& dep = SDep(SU, SDep::Data, DataLatency, *Alias);
339 if (!UnitLatencies) {
Dan Gohman3fb150a2010-04-17 17:42:52 +0000340 ComputeOperandLatency(SU, UseSU, const_cast<SDep &>(dep));
341 ST.adjustSchedDependency(SU, UseSU, const_cast<SDep &>(dep));
David Goodwin71046162009-08-13 16:05:04 +0000342 }
Evan Chenga69ec092010-03-22 21:24:33 +0000343 UseSU->addPred(dep);
Dan Gohman3f237442008-12-16 03:25:46 +0000344 }
Dan Gohman343f0c02008-11-19 23:18:57 +0000345 }
346
Dan Gohman8749b612008-12-16 03:35:01 +0000347 // If a def is going to wrap back around to the top of the loop,
348 // backschedule it.
Dan Gohman9e64bbb2009-02-10 23:27:53 +0000349 if (!UnitLatencies && DefList.empty()) {
Dan Gohman8749b612008-12-16 03:35:01 +0000350 LoopDependencies::LoopDeps::iterator I = LoopRegs.Deps.find(Reg);
351 if (I != LoopRegs.Deps.end()) {
352 const MachineOperand *UseMO = I->second.first;
353 unsigned Count = I->second.second;
354 const MachineInstr *UseMI = UseMO->getParent();
355 unsigned UseMOIdx = UseMO - &UseMI->getOperand(0);
Evan Chenge837dea2011-06-28 19:10:37 +0000356 const MCInstrDesc &UseMCID = UseMI->getDesc();
Dan Gohman8749b612008-12-16 03:35:01 +0000357 // TODO: If we knew the total depth of the region here, we could
358 // handle the case where the whole loop is inside the region but
359 // is large enough that the isScheduleHigh trick isn't needed.
Evan Chenge837dea2011-06-28 19:10:37 +0000360 if (UseMOIdx < UseMCID.getNumOperands()) {
Dan Gohman8749b612008-12-16 03:35:01 +0000361 // Currently, we only support scheduling regions consisting of
362 // single basic blocks. Check to see if the instruction is in
363 // the same region by checking to see if it has the same parent.
364 if (UseMI->getParent() != MI->getParent()) {
365 unsigned Latency = SU->Latency;
Evan Chenge837dea2011-06-28 19:10:37 +0000366 if (UseMCID.OpInfo[UseMOIdx].isLookupPtrRegClass())
Dan Gohman8749b612008-12-16 03:35:01 +0000367 Latency += SpecialAddressLatency;
368 // This is a wild guess as to the portion of the latency which
369 // will be overlapped by work done outside the current
370 // scheduling region.
371 Latency -= std::min(Latency, Count);
Chris Lattner7a2bdde2011-04-15 05:18:47 +0000372 // Add the artificial edge.
Dan Gohman9e64bbb2009-02-10 23:27:53 +0000373 ExitSU.addPred(SDep(SU, SDep::Order, Latency,
374 /*Reg=*/0, /*isNormalMemory=*/false,
375 /*isMustAlias=*/false,
376 /*isArtificial=*/true));
Dan Gohman8749b612008-12-16 03:35:01 +0000377 } else if (SpecialAddressLatency > 0 &&
Evan Chenge837dea2011-06-28 19:10:37 +0000378 UseMCID.OpInfo[UseMOIdx].isLookupPtrRegClass()) {
Dan Gohman8749b612008-12-16 03:35:01 +0000379 // The entire loop body is within the current scheduling region
380 // and the latency of this operation is assumed to be greater
381 // than the latency of the loop.
382 // TODO: Recursively mark data-edge predecessors as
383 // isScheduleHigh too.
384 SU->isScheduleHigh = true;
385 }
386 }
387 LoopRegs.Deps.erase(I);
388 }
389 }
390
Dan Gohman343f0c02008-11-19 23:18:57 +0000391 UseList.clear();
Dan Gohman3f237442008-12-16 03:25:46 +0000392 if (!MO.isDead())
393 DefList.clear();
Andrew Trickee109152011-05-05 19:32:21 +0000394
395 // Calls will not be reordered because of chain dependencies (see
396 // below). Since call operands are dead, calls may continue to be added
397 // to the DefList making dependence checking quadratic in the size of
398 // the block. Instead, we leave only one call at the back of the
399 // DefList.
Andrew Trickee109152011-05-05 19:32:21 +0000400 if (SU->isCall) {
401 while (!DefList.empty() && DefList.back()->isCall)
402 DefList.pop_back();
403 }
Dan Gohman3f237442008-12-16 03:25:46 +0000404 DefList.push_back(SU);
Dan Gohman343f0c02008-11-19 23:18:57 +0000405 } else {
406 UseList.push_back(SU);
407 }
408 }
Dan Gohman6a9041e2008-12-04 01:35:46 +0000409
410 // Add chain dependencies.
David Goodwin7c9b1ac2009-11-02 17:06:28 +0000411 // Chain dependencies used to enforce memory order should have
412 // latency of 0 (except for true dependency of Store followed by
413 // aliased Load... we estimate that with a single cycle of latency
414 // assuming the hardware will bypass)
Dan Gohman6a9041e2008-12-04 01:35:46 +0000415 // Note that isStoreToStackSlot and isLoadFromStackSLot are not usable
416 // after stack slots are lowered to actual addresses.
417 // TODO: Use an AliasAnalysis and do real alias-analysis queries, and
418 // produce more precise dependence information.
David Goodwin7c9b1ac2009-11-02 17:06:28 +0000419#define STORE_LOAD_LATENCY 1
420 unsigned TrueMemOrderLatency = 0;
Evan Chenge837dea2011-06-28 19:10:37 +0000421 if (MCID.isCall() || MI->hasUnmodeledSideEffects() ||
Andrew Trickf405b1a2011-05-05 19:24:06 +0000422 (MI->hasVolatileMemoryRef() &&
Evan Chenge837dea2011-06-28 19:10:37 +0000423 (!MCID.mayLoad() || !MI->isInvariantLoad(AA)))) {
David Goodwin980d4942009-11-09 19:22:17 +0000424 // Be conservative with these and add dependencies on all memory
425 // references, even those that are known to not alias.
Andrew Trickf405b1a2011-05-05 19:24:06 +0000426 for (std::map<const Value *, SUnit *>::iterator I =
David Goodwin980d4942009-11-09 19:22:17 +0000427 NonAliasMemDefs.begin(), E = NonAliasMemDefs.end(); I != E; ++I) {
David Goodwin7c9b1ac2009-11-02 17:06:28 +0000428 I->second->addPred(SDep(SU, SDep::Order, /*Latency=*/0));
Dan Gohman6a9041e2008-12-04 01:35:46 +0000429 }
430 for (std::map<const Value *, std::vector<SUnit *> >::iterator I =
David Goodwin980d4942009-11-09 19:22:17 +0000431 NonAliasMemUses.begin(), E = NonAliasMemUses.end(); I != E; ++I) {
Dan Gohman6a9041e2008-12-04 01:35:46 +0000432 for (unsigned i = 0, e = I->second.size(); i != e; ++i)
David Goodwin7c9b1ac2009-11-02 17:06:28 +0000433 I->second[i]->addPred(SDep(SU, SDep::Order, TrueMemOrderLatency));
Dan Gohman6a9041e2008-12-04 01:35:46 +0000434 }
David Goodwin980d4942009-11-09 19:22:17 +0000435 NonAliasMemDefs.clear();
436 NonAliasMemUses.clear();
437 // Add SU to the barrier chain.
438 if (BarrierChain)
439 BarrierChain->addPred(SDep(SU, SDep::Order, /*Latency=*/0));
440 BarrierChain = SU;
441
442 // fall-through
443 new_alias_chain:
444 // Chain all possibly aliasing memory references though SU.
445 if (AliasChain)
446 AliasChain->addPred(SDep(SU, SDep::Order, /*Latency=*/0));
447 AliasChain = SU;
448 for (unsigned k = 0, m = PendingLoads.size(); k != m; ++k)
449 PendingLoads[k]->addPred(SDep(SU, SDep::Order, TrueMemOrderLatency));
450 for (std::map<const Value *, SUnit *>::iterator I = AliasMemDefs.begin(),
451 E = AliasMemDefs.end(); I != E; ++I) {
452 I->second->addPred(SDep(SU, SDep::Order, /*Latency=*/0));
453 }
454 for (std::map<const Value *, std::vector<SUnit *> >::iterator I =
455 AliasMemUses.begin(), E = AliasMemUses.end(); I != E; ++I) {
456 for (unsigned i = 0, e = I->second.size(); i != e; ++i)
457 I->second[i]->addPred(SDep(SU, SDep::Order, TrueMemOrderLatency));
458 }
459 PendingLoads.clear();
460 AliasMemDefs.clear();
461 AliasMemUses.clear();
Evan Chenge837dea2011-06-28 19:10:37 +0000462 } else if (MCID.mayStore()) {
David Goodwina9e61072009-11-03 20:15:00 +0000463 bool MayAlias = true;
David Goodwin7c9b1ac2009-11-02 17:06:28 +0000464 TrueMemOrderLatency = STORE_LOAD_LATENCY;
David Goodwina9e61072009-11-03 20:15:00 +0000465 if (const Value *V = getUnderlyingObjectForInstr(MI, MFI, MayAlias)) {
Dan Gohman6a9041e2008-12-04 01:35:46 +0000466 // A store to a specific PseudoSourceValue. Add precise dependencies.
David Goodwin980d4942009-11-09 19:22:17 +0000467 // Record the def in MemDefs, first adding a dep if there is
468 // an existing def.
Andrew Trickf405b1a2011-05-05 19:24:06 +0000469 std::map<const Value *, SUnit *>::iterator I =
David Goodwin980d4942009-11-09 19:22:17 +0000470 ((MayAlias) ? AliasMemDefs.find(V) : NonAliasMemDefs.find(V));
Andrew Trickf405b1a2011-05-05 19:24:06 +0000471 std::map<const Value *, SUnit *>::iterator IE =
David Goodwin980d4942009-11-09 19:22:17 +0000472 ((MayAlias) ? AliasMemDefs.end() : NonAliasMemDefs.end());
473 if (I != IE) {
David Goodwin7c9b1ac2009-11-02 17:06:28 +0000474 I->second->addPred(SDep(SU, SDep::Order, /*Latency=*/0, /*Reg=*/0,
Dan Gohman54e4c362008-12-09 22:54:47 +0000475 /*isNormalMemory=*/true));
Dan Gohman6a9041e2008-12-04 01:35:46 +0000476 I->second = SU;
477 } else {
David Goodwin980d4942009-11-09 19:22:17 +0000478 if (MayAlias)
479 AliasMemDefs[V] = SU;
480 else
481 NonAliasMemDefs[V] = SU;
Dan Gohman6a9041e2008-12-04 01:35:46 +0000482 }
483 // Handle the uses in MemUses, if there are any.
Dan Gohmana629b482008-12-08 17:50:35 +0000484 std::map<const Value *, std::vector<SUnit *> >::iterator J =
David Goodwin980d4942009-11-09 19:22:17 +0000485 ((MayAlias) ? AliasMemUses.find(V) : NonAliasMemUses.find(V));
486 std::map<const Value *, std::vector<SUnit *> >::iterator JE =
487 ((MayAlias) ? AliasMemUses.end() : NonAliasMemUses.end());
488 if (J != JE) {
Dan Gohman6a9041e2008-12-04 01:35:46 +0000489 for (unsigned i = 0, e = J->second.size(); i != e; ++i)
David Goodwin7c9b1ac2009-11-02 17:06:28 +0000490 J->second[i]->addPred(SDep(SU, SDep::Order, TrueMemOrderLatency,
491 /*Reg=*/0, /*isNormalMemory=*/true));
Dan Gohman6a9041e2008-12-04 01:35:46 +0000492 J->second.clear();
493 }
David Goodwina9e61072009-11-03 20:15:00 +0000494 if (MayAlias) {
David Goodwin980d4942009-11-09 19:22:17 +0000495 // Add dependencies from all the PendingLoads, i.e. loads
496 // with no underlying object.
David Goodwina9e61072009-11-03 20:15:00 +0000497 for (unsigned k = 0, m = PendingLoads.size(); k != m; ++k)
498 PendingLoads[k]->addPred(SDep(SU, SDep::Order, TrueMemOrderLatency));
David Goodwin980d4942009-11-09 19:22:17 +0000499 // Add dependence on alias chain, if needed.
500 if (AliasChain)
501 AliasChain->addPred(SDep(SU, SDep::Order, /*Latency=*/0));
David Goodwina9e61072009-11-03 20:15:00 +0000502 }
David Goodwin980d4942009-11-09 19:22:17 +0000503 // Add dependence on barrier chain, if needed.
504 if (BarrierChain)
505 BarrierChain->addPred(SDep(SU, SDep::Order, /*Latency=*/0));
David Goodwin5be870a2009-11-05 00:16:44 +0000506 } else {
Dan Gohman6a9041e2008-12-04 01:35:46 +0000507 // Treat all other stores conservatively.
David Goodwin980d4942009-11-09 19:22:17 +0000508 goto new_alias_chain;
David Goodwin7c9b1ac2009-11-02 17:06:28 +0000509 }
Evan Chengec6906b2010-10-23 02:10:46 +0000510
511 if (!ExitSU.isPred(SU))
512 // Push store's up a bit to avoid them getting in between cmp
513 // and branches.
514 ExitSU.addPred(SDep(SU, SDep::Order, 0,
515 /*Reg=*/0, /*isNormalMemory=*/false,
516 /*isMustAlias=*/false,
517 /*isArtificial=*/true));
Evan Chenge837dea2011-06-28 19:10:37 +0000518 } else if (MCID.mayLoad()) {
David Goodwina9e61072009-11-03 20:15:00 +0000519 bool MayAlias = true;
David Goodwin7c9b1ac2009-11-02 17:06:28 +0000520 TrueMemOrderLatency = 0;
Dan Gohmana70dca12009-10-09 23:27:56 +0000521 if (MI->isInvariantLoad(AA)) {
Dan Gohman6a9041e2008-12-04 01:35:46 +0000522 // Invariant load, no chain dependencies needed!
David Goodwin5be870a2009-11-05 00:16:44 +0000523 } else {
Andrew Trickf405b1a2011-05-05 19:24:06 +0000524 if (const Value *V =
David Goodwin980d4942009-11-09 19:22:17 +0000525 getUnderlyingObjectForInstr(MI, MFI, MayAlias)) {
526 // A load from a specific PseudoSourceValue. Add precise dependencies.
Andrew Trickf405b1a2011-05-05 19:24:06 +0000527 std::map<const Value *, SUnit *>::iterator I =
David Goodwin980d4942009-11-09 19:22:17 +0000528 ((MayAlias) ? AliasMemDefs.find(V) : NonAliasMemDefs.find(V));
Andrew Trickf405b1a2011-05-05 19:24:06 +0000529 std::map<const Value *, SUnit *>::iterator IE =
David Goodwin980d4942009-11-09 19:22:17 +0000530 ((MayAlias) ? AliasMemDefs.end() : NonAliasMemDefs.end());
531 if (I != IE)
532 I->second->addPred(SDep(SU, SDep::Order, /*Latency=*/0, /*Reg=*/0,
533 /*isNormalMemory=*/true));
534 if (MayAlias)
535 AliasMemUses[V].push_back(SU);
Andrew Trickf405b1a2011-05-05 19:24:06 +0000536 else
David Goodwin980d4942009-11-09 19:22:17 +0000537 NonAliasMemUses[V].push_back(SU);
538 } else {
539 // A load with no underlying object. Depend on all
540 // potentially aliasing stores.
Andrew Trickf405b1a2011-05-05 19:24:06 +0000541 for (std::map<const Value *, SUnit *>::iterator I =
David Goodwin980d4942009-11-09 19:22:17 +0000542 AliasMemDefs.begin(), E = AliasMemDefs.end(); I != E; ++I)
543 I->second->addPred(SDep(SU, SDep::Order, /*Latency=*/0));
Andrew Trickf405b1a2011-05-05 19:24:06 +0000544
David Goodwin980d4942009-11-09 19:22:17 +0000545 PendingLoads.push_back(SU);
546 MayAlias = true;
David Goodwina9e61072009-11-03 20:15:00 +0000547 }
Andrew Trickf405b1a2011-05-05 19:24:06 +0000548
David Goodwin980d4942009-11-09 19:22:17 +0000549 // Add dependencies on alias and barrier chains, if needed.
550 if (MayAlias && AliasChain)
551 AliasChain->addPred(SDep(SU, SDep::Order, /*Latency=*/0));
552 if (BarrierChain)
553 BarrierChain->addPred(SDep(SU, SDep::Order, /*Latency=*/0));
Andrew Trickf405b1a2011-05-05 19:24:06 +0000554 }
Dan Gohman343f0c02008-11-19 23:18:57 +0000555 }
Dan Gohman343f0c02008-11-19 23:18:57 +0000556 }
Devang Patelcf4cc842011-06-02 20:07:12 +0000557 if (PrevMI)
558 FirstDbgValue = PrevMI;
Dan Gohman79ce2762009-01-15 19:20:50 +0000559
560 for (int i = 0, e = TRI->getNumRegs(); i != e; ++i) {
561 Defs[i].clear();
562 Uses[i].clear();
563 }
564 PendingLoads.clear();
Dan Gohman343f0c02008-11-19 23:18:57 +0000565}
566
Dan Gohman9e64bbb2009-02-10 23:27:53 +0000567void ScheduleDAGInstrs::FinishBlock() {
568 // Nothing to do.
569}
570
Dan Gohmanc8c28272008-11-21 00:12:10 +0000571void ScheduleDAGInstrs::ComputeLatency(SUnit *SU) {
David Goodwind94a4e52009-08-10 15:55:25 +0000572 // Compute the latency for the node.
Evan Cheng3ef1c872010-09-10 01:29:16 +0000573 if (!InstrItins || InstrItins->isEmpty()) {
574 SU->Latency = 1;
Dan Gohman4ea8e852008-12-16 02:38:22 +0000575
Evan Cheng3ef1c872010-09-10 01:29:16 +0000576 // Simplistic target-independent heuristic: assume that loads take
577 // extra time.
Dan Gohman4ea8e852008-12-16 02:38:22 +0000578 if (SU->getInstr()->getDesc().mayLoad())
579 SU->Latency += 2;
Evan Cheng8239daf2010-11-03 00:45:17 +0000580 } else {
581 SU->Latency = TII->getInstrLatency(InstrItins, SU->getInstr());
582 }
Dan Gohmanc8c28272008-11-21 00:12:10 +0000583}
584
Andrew Trickf405b1a2011-05-05 19:24:06 +0000585void ScheduleDAGInstrs::ComputeOperandLatency(SUnit *Def, SUnit *Use,
David Goodwindc4bdcd2009-08-19 16:08:58 +0000586 SDep& dep) const {
Evan Cheng3ef1c872010-09-10 01:29:16 +0000587 if (!InstrItins || InstrItins->isEmpty())
David Goodwindc4bdcd2009-08-19 16:08:58 +0000588 return;
Andrew Trickf405b1a2011-05-05 19:24:06 +0000589
David Goodwindc4bdcd2009-08-19 16:08:58 +0000590 // For a data dependency with a known register...
591 if ((dep.getKind() != SDep::Data) || (dep.getReg() == 0))
592 return;
593
594 const unsigned Reg = dep.getReg();
595
596 // ... find the definition of the register in the defining
597 // instruction
598 MachineInstr *DefMI = Def->getInstr();
599 int DefIdx = DefMI->findRegisterDefOperandIdx(Reg);
600 if (DefIdx != -1) {
Evan Cheng1aca5bc2010-10-08 18:42:25 +0000601 const MachineOperand &MO = DefMI->getOperand(DefIdx);
602 if (MO.isReg() && MO.isImplicit() &&
Evan Chengd82de832010-10-08 23:01:57 +0000603 DefIdx >= (int)DefMI->getDesc().getNumOperands()) {
Evan Cheng1aca5bc2010-10-08 18:42:25 +0000604 // This is an implicit def, getOperandLatency() won't return the correct
605 // latency. e.g.
606 // %D6<def>, %D7<def> = VLD1q16 %R2<kill>, 0, ..., %Q3<imp-def>
607 // %Q1<def> = VMULv8i16 %Q1<kill>, %Q3<kill>, ...
608 // What we want is to compute latency between def of %D6/%D7 and use of
609 // %Q3 instead.
610 DefIdx = DefMI->findRegisterDefOperandIdx(Reg, false, true, TRI);
611 }
Evan Chenga0792de2010-10-06 06:27:31 +0000612 MachineInstr *UseMI = Use->getInstr();
Evan Cheng3881cb72010-09-29 22:42:35 +0000613 // For all uses of the register, calculate the maxmimum latency
614 int Latency = -1;
Evan Chengec6906b2010-10-23 02:10:46 +0000615 if (UseMI) {
616 for (unsigned i = 0, e = UseMI->getNumOperands(); i != e; ++i) {
617 const MachineOperand &MO = UseMI->getOperand(i);
618 if (!MO.isReg() || !MO.isUse())
619 continue;
620 unsigned MOReg = MO.getReg();
621 if (MOReg != Reg)
622 continue;
David Goodwindc4bdcd2009-08-19 16:08:58 +0000623
Evan Chengec6906b2010-10-23 02:10:46 +0000624 int UseCycle = TII->getOperandLatency(InstrItins, DefMI, DefIdx,
625 UseMI, i);
626 Latency = std::max(Latency, UseCycle);
627 }
628 } else {
629 // UseMI is null, then it must be a scheduling barrier.
630 if (!InstrItins || InstrItins->isEmpty())
631 return;
632 unsigned DefClass = DefMI->getDesc().getSchedClass();
633 Latency = InstrItins->getOperandCycle(DefClass, DefIdx);
David Goodwindc4bdcd2009-08-19 16:08:58 +0000634 }
Evan Chengec6906b2010-10-23 02:10:46 +0000635
636 // If we found a latency, then replace the existing dependence latency.
637 if (Latency >= 0)
638 dep.setLatency(Latency);
David Goodwindc4bdcd2009-08-19 16:08:58 +0000639 }
640}
641
Dan Gohman343f0c02008-11-19 23:18:57 +0000642void ScheduleDAGInstrs::dumpNode(const SUnit *SU) const {
643 SU->getInstr()->dump();
644}
645
646std::string ScheduleDAGInstrs::getGraphNodeLabel(const SUnit *SU) const {
647 std::string s;
648 raw_string_ostream oss(s);
Dan Gohman9e64bbb2009-02-10 23:27:53 +0000649 if (SU == &EntrySU)
650 oss << "<entry>";
651 else if (SU == &ExitSU)
652 oss << "<exit>";
653 else
654 SU->getInstr()->print(oss);
Dan Gohman343f0c02008-11-19 23:18:57 +0000655 return oss.str();
656}
657
658// EmitSchedule - Emit the machine code in scheduled order.
Dan Gohmanaf1d8ca2010-05-01 00:01:06 +0000659MachineBasicBlock *ScheduleDAGInstrs::EmitSchedule() {
Dan Gohman343f0c02008-11-19 23:18:57 +0000660 // For MachineInstr-based scheduling, we're rescheduling the instructions in
661 // the block, so start by removing them from the block.
Dan Gohman47ac0f02009-02-11 04:27:20 +0000662 while (Begin != InsertPos) {
Dan Gohmanf7119392009-01-16 22:10:20 +0000663 MachineBasicBlock::iterator I = Begin;
664 ++Begin;
665 BB->remove(I);
666 }
Dan Gohman343f0c02008-11-19 23:18:57 +0000667
Devang Patelcf4cc842011-06-02 20:07:12 +0000668 // If first instruction was a DBG_VALUE then put it back.
669 if (FirstDbgValue)
670 BB->insert(InsertPos, FirstDbgValue);
Dale Johannesenbfdf7f32010-03-10 22:13:47 +0000671
Dan Gohman0b1d4a72008-12-23 21:37:04 +0000672 // Then re-insert them according to the given schedule.
Dan Gohman343f0c02008-11-19 23:18:57 +0000673 for (unsigned i = 0, e = Sequence.size(); i != e; i++) {
Devang Patelee1f8782011-06-02 21:31:00 +0000674 if (SUnit *SU = Sequence[i])
675 BB->insert(InsertPos, SU->getInstr());
676 else
Dan Gohman343f0c02008-11-19 23:18:57 +0000677 // Null SUnit* is a noop.
678 EmitNoop();
Dan Gohman343f0c02008-11-19 23:18:57 +0000679 }
680
Dan Gohman9e64bbb2009-02-10 23:27:53 +0000681 // Update the Begin iterator, as the first instruction in the block
682 // may have been scheduled later.
Devang Patelcf4cc842011-06-02 20:07:12 +0000683 if (!Sequence.empty())
Dan Gohman9e64bbb2009-02-10 23:27:53 +0000684 Begin = Sequence[0]->getInstr();
685
Devang Patelcf4cc842011-06-02 20:07:12 +0000686 // Reinsert any remaining debug_values.
687 for (std::vector<std::pair<MachineInstr *, MachineInstr *> >::iterator
688 DI = DbgValues.end(), DE = DbgValues.begin(); DI != DE; --DI) {
689 std::pair<MachineInstr *, MachineInstr *> P = *prior(DI);
690 MachineInstr *DbgValue = P.first;
691 MachineInstr *OrigPrivMI = P.second;
692 BB->insertAfter(OrigPrivMI, DbgValue);
693 }
694 DbgValues.clear();
695 FirstDbgValue = NULL;
Dan Gohman343f0c02008-11-19 23:18:57 +0000696 return BB;
697}