Dale Johannesen | 4dc35db | 2007-07-13 17:31:29 +0000 | [diff] [blame] | 1 | //===----- SchedulePostRAList.cpp - list scheduler ------------------------===// |
Dale Johannesen | 2182f06 | 2007-07-13 17:13:54 +0000 | [diff] [blame] | 2 | // |
| 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
Chris Lattner | f3ebc3f | 2007-12-29 20:36:04 +0000 | [diff] [blame] | 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
Dale Johannesen | 2182f06 | 2007-07-13 17:13:54 +0000 | [diff] [blame] | 7 | // |
| 8 | //===----------------------------------------------------------------------===// |
| 9 | // |
| 10 | // This implements a top-down list scheduler, using standard algorithms. |
| 11 | // The basic approach uses a priority queue of available nodes to schedule. |
| 12 | // One at a time, nodes are taken from the priority queue (thus in priority |
| 13 | // order), checked for legality to schedule, and emitted if legal. |
| 14 | // |
| 15 | // Nodes may not be legal to schedule either due to structural hazards (e.g. |
| 16 | // pipeline or resource constraints) or because an input to the instruction has |
| 17 | // not completed execution. |
| 18 | // |
| 19 | //===----------------------------------------------------------------------===// |
| 20 | |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 21 | #include "AggressiveAntiDepBreaker.h" |
| 22 | #include "AntiDepBreaker.h" |
| 23 | #include "CriticalAntiDepBreaker.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 24 | #include "llvm/ADT/Statistic.h" |
| 25 | #include "llvm/Analysis/AliasAnalysis.h" |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 26 | #include "llvm/CodeGen/LatencyPriorityQueue.h" |
Dan Gohman | dddc1ac | 2008-12-16 03:25:46 +0000 | [diff] [blame] | 27 | #include "llvm/CodeGen/MachineDominators.h" |
David Goodwin | be3039e | 2009-10-01 19:45:32 +0000 | [diff] [blame] | 28 | #include "llvm/CodeGen/MachineFrameInfo.h" |
Dale Johannesen | 2182f06 | 2007-07-13 17:13:54 +0000 | [diff] [blame] | 29 | #include "llvm/CodeGen/MachineFunctionPass.h" |
Dan Gohman | dddc1ac | 2008-12-16 03:25:46 +0000 | [diff] [blame] | 30 | #include "llvm/CodeGen/MachineLoopInfo.h" |
Dan Gohman | ad2134d | 2008-11-25 00:52:40 +0000 | [diff] [blame] | 31 | #include "llvm/CodeGen/MachineRegisterInfo.h" |
Mehdi Amini | b550cb1 | 2016-04-18 09:17:29 +0000 | [diff] [blame] | 32 | #include "llvm/CodeGen/Passes.h" |
Andrew Trick | 05ff466 | 2012-06-06 20:29:31 +0000 | [diff] [blame] | 33 | #include "llvm/CodeGen/RegisterClassInfo.h" |
Andrew Trick | 9a0c583 | 2012-03-07 23:01:06 +0000 | [diff] [blame] | 34 | #include "llvm/CodeGen/ScheduleDAGInstrs.h" |
Dan Gohman | ceac7c3 | 2009-01-16 01:33:36 +0000 | [diff] [blame] | 35 | #include "llvm/CodeGen/ScheduleHazardRecognizer.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 36 | #include "llvm/CodeGen/SchedulerRegistry.h" |
Matthias Braun | 31d19d4 | 2016-05-10 03:21:59 +0000 | [diff] [blame] | 37 | #include "llvm/CodeGen/TargetPassConfig.h" |
David Goodwin | e056d10 | 2009-10-26 22:31:16 +0000 | [diff] [blame] | 38 | #include "llvm/Support/CommandLine.h" |
Dale Johannesen | 2182f06 | 2007-07-13 17:13:54 +0000 | [diff] [blame] | 39 | #include "llvm/Support/Debug.h" |
Torok Edwin | 56d0659 | 2009-07-11 20:10:48 +0000 | [diff] [blame] | 40 | #include "llvm/Support/ErrorHandling.h" |
David Goodwin | f20236a | 2009-08-11 01:44:26 +0000 | [diff] [blame] | 41 | #include "llvm/Support/raw_ostream.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 42 | #include "llvm/Target/TargetInstrInfo.h" |
| 43 | #include "llvm/Target/TargetLowering.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 44 | #include "llvm/Target/TargetRegisterInfo.h" |
| 45 | #include "llvm/Target/TargetSubtargetInfo.h" |
Dale Johannesen | 2182f06 | 2007-07-13 17:13:54 +0000 | [diff] [blame] | 46 | using namespace llvm; |
| 47 | |
Chandler Carruth | 1b9dde0 | 2014-04-22 02:02:50 +0000 | [diff] [blame] | 48 | #define DEBUG_TYPE "post-RA-sched" |
| 49 | |
Dan Gohman | ceac7c3 | 2009-01-16 01:33:36 +0000 | [diff] [blame] | 50 | STATISTIC(NumNoops, "Number of noops inserted"); |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 51 | STATISTIC(NumStalls, "Number of pipeline stalls"); |
David Goodwin | 8370485 | 2009-10-26 16:59:04 +0000 | [diff] [blame] | 52 | STATISTIC(NumFixedAnti, "Number of fixed anti-dependencies"); |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 53 | |
David Goodwin | 9a051a5 | 2009-10-01 21:46:35 +0000 | [diff] [blame] | 54 | // Post-RA scheduling is enabled with |
Evan Cheng | 0d639a2 | 2011-07-01 21:01:15 +0000 | [diff] [blame] | 55 | // TargetSubtargetInfo.enablePostRAScheduler(). This flag can be used to |
David Goodwin | 9a051a5 | 2009-10-01 21:46:35 +0000 | [diff] [blame] | 56 | // override the target. |
| 57 | static cl::opt<bool> |
| 58 | EnablePostRAScheduler("post-RA-scheduler", |
| 59 | cl::desc("Enable scheduling after register allocation"), |
David Goodwin | 1cc6dd9 | 2009-10-01 22:19:57 +0000 | [diff] [blame] | 60 | cl::init(false), cl::Hidden); |
David Goodwin | 8370485 | 2009-10-26 16:59:04 +0000 | [diff] [blame] | 61 | static cl::opt<std::string> |
Dan Gohman | ad2134d | 2008-11-25 00:52:40 +0000 | [diff] [blame] | 62 | EnableAntiDepBreaking("break-anti-dependencies", |
David Goodwin | 8370485 | 2009-10-26 16:59:04 +0000 | [diff] [blame] | 63 | cl::desc("Break post-RA scheduling anti-dependencies: " |
| 64 | "\"critical\", \"all\", or \"none\""), |
| 65 | cl::init("none"), cl::Hidden); |
Dan Gohman | ceac7c3 | 2009-01-16 01:33:36 +0000 | [diff] [blame] | 66 | |
David Goodwin | 7f65169 | 2009-09-01 18:34:03 +0000 | [diff] [blame] | 67 | // If DebugDiv > 0 then only schedule MBB with (ID % DebugDiv) == DebugMod |
| 68 | static cl::opt<int> |
| 69 | DebugDiv("postra-sched-debugdiv", |
| 70 | cl::desc("Debug control MBBs that are scheduled"), |
| 71 | cl::init(0), cl::Hidden); |
| 72 | static cl::opt<int> |
| 73 | DebugMod("postra-sched-debugmod", |
| 74 | cl::desc("Debug control MBBs that are scheduled"), |
| 75 | cl::init(0), cl::Hidden); |
| 76 | |
David Goodwin | 661ea98 | 2009-10-26 19:41:00 +0000 | [diff] [blame] | 77 | AntiDepBreaker::~AntiDepBreaker() { } |
| 78 | |
Dale Johannesen | 2182f06 | 2007-07-13 17:13:54 +0000 | [diff] [blame] | 79 | namespace { |
Nick Lewycky | 02d5f77 | 2009-10-25 06:33:48 +0000 | [diff] [blame] | 80 | class PostRAScheduler : public MachineFunctionPass { |
Evan Cheng | 2d51c7c | 2010-06-18 23:09:54 +0000 | [diff] [blame] | 81 | const TargetInstrInfo *TII; |
Jakob Stoklund Olesen | 4f5f84c | 2011-06-16 21:56:21 +0000 | [diff] [blame] | 82 | RegisterClassInfo RegClassInfo; |
Dan Gohman | 87b02d5 | 2009-10-09 23:27:56 +0000 | [diff] [blame] | 83 | |
Dale Johannesen | 2182f06 | 2007-07-13 17:13:54 +0000 | [diff] [blame] | 84 | public: |
| 85 | static char ID; |
Andrew Trick | df7e376 | 2012-02-08 21:22:53 +0000 | [diff] [blame] | 86 | PostRAScheduler() : MachineFunctionPass(ID) {} |
Dan Gohman | ad2134d | 2008-11-25 00:52:40 +0000 | [diff] [blame] | 87 | |
Craig Topper | 4584cd5 | 2014-03-07 09:26:03 +0000 | [diff] [blame] | 88 | void getAnalysisUsage(AnalysisUsage &AU) const override { |
Dan Gohman | 0402315 | 2009-07-31 23:37:33 +0000 | [diff] [blame] | 89 | AU.setPreservesCFG(); |
Chandler Carruth | 7b560d4 | 2015-09-09 17:55:00 +0000 | [diff] [blame] | 90 | AU.addRequired<AAResultsWrapperPass>(); |
Andrew Trick | df7e376 | 2012-02-08 21:22:53 +0000 | [diff] [blame] | 91 | AU.addRequired<TargetPassConfig>(); |
Dan Gohman | dddc1ac | 2008-12-16 03:25:46 +0000 | [diff] [blame] | 92 | AU.addRequired<MachineDominatorTree>(); |
| 93 | AU.addPreserved<MachineDominatorTree>(); |
| 94 | AU.addRequired<MachineLoopInfo>(); |
| 95 | AU.addPreserved<MachineLoopInfo>(); |
| 96 | MachineFunctionPass::getAnalysisUsage(AU); |
| 97 | } |
| 98 | |
Derek Schuff | ad154c8 | 2016-03-28 17:05:30 +0000 | [diff] [blame] | 99 | MachineFunctionProperties getRequiredProperties() const override { |
| 100 | return MachineFunctionProperties().set( |
Matthias Braun | 1eb4736 | 2016-08-25 01:27:13 +0000 | [diff] [blame] | 101 | MachineFunctionProperties::Property::NoVRegs); |
Derek Schuff | ad154c8 | 2016-03-28 17:05:30 +0000 | [diff] [blame] | 102 | } |
| 103 | |
Craig Topper | 4584cd5 | 2014-03-07 09:26:03 +0000 | [diff] [blame] | 104 | bool runOnMachineFunction(MachineFunction &Fn) override; |
NAKAMURA Takumi | f51a34e | 2014-10-29 15:23:11 +0000 | [diff] [blame] | 105 | |
Mitch Bodart | 6453501 | 2016-05-19 16:40:49 +0000 | [diff] [blame] | 106 | private: |
Sanjay Patel | a2f658d | 2014-07-15 22:39:58 +0000 | [diff] [blame] | 107 | bool enablePostRAScheduler( |
| 108 | const TargetSubtargetInfo &ST, CodeGenOpt::Level OptLevel, |
| 109 | TargetSubtargetInfo::AntiDepBreakMode &Mode, |
| 110 | TargetSubtargetInfo::RegClassVector &CriticalPathRCs) const; |
Dale Johannesen | 2182f06 | 2007-07-13 17:13:54 +0000 | [diff] [blame] | 111 | }; |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 112 | char PostRAScheduler::ID = 0; |
| 113 | |
Nick Lewycky | 02d5f77 | 2009-10-25 06:33:48 +0000 | [diff] [blame] | 114 | class SchedulePostRATDList : public ScheduleDAGInstrs { |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 115 | /// AvailableQueue - The priority queue to use for the available SUnits. |
Dan Gohman | 682a2d1 | 2009-10-21 01:44:44 +0000 | [diff] [blame] | 116 | /// |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 117 | LatencyPriorityQueue AvailableQueue; |
Jim Grosbach | d772bde | 2010-05-14 21:19:48 +0000 | [diff] [blame] | 118 | |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 119 | /// PendingQueue - This contains all of the instructions whose operands have |
| 120 | /// been issued, but their results are not ready yet (due to the latency of |
| 121 | /// the operation). Once the operands becomes available, the instruction is |
| 122 | /// added to the AvailableQueue. |
| 123 | std::vector<SUnit*> PendingQueue; |
| 124 | |
Dan Gohman | ceac7c3 | 2009-01-16 01:33:36 +0000 | [diff] [blame] | 125 | /// HazardRec - The hazard recognizer to use. |
| 126 | ScheduleHazardRecognizer *HazardRec; |
| 127 | |
David Goodwin | 8370485 | 2009-10-26 16:59:04 +0000 | [diff] [blame] | 128 | /// AntiDepBreak - Anti-dependence breaking object, or NULL if none |
| 129 | AntiDepBreaker *AntiDepBreak; |
| 130 | |
Dan Gohman | 87b02d5 | 2009-10-09 23:27:56 +0000 | [diff] [blame] | 131 | /// AA - AliasAnalysis for making memory reference queries. |
| 132 | AliasAnalysis *AA; |
| 133 | |
Andrew Trick | 60cf03e | 2012-03-07 05:21:52 +0000 | [diff] [blame] | 134 | /// The schedule. Null SUnit*'s represent noop instructions. |
| 135 | std::vector<SUnit*> Sequence; |
| 136 | |
Krzysztof Parzyszek | 5c61d11 | 2016-03-05 15:45:23 +0000 | [diff] [blame] | 137 | /// Ordered list of DAG postprocessing steps. |
| 138 | std::vector<std::unique_ptr<ScheduleDAGMutation>> Mutations; |
| 139 | |
Andrew Trick | a53e101 | 2013-08-23 17:48:33 +0000 | [diff] [blame] | 140 | /// The index in BB of RegionEnd. |
| 141 | /// |
| 142 | /// This is the instruction number from the top of the current block, not |
| 143 | /// the SlotIndex. It is only used by the AntiDepBreaker. |
| 144 | unsigned EndIndex; |
| 145 | |
Dan Gohman | ad2134d | 2008-11-25 00:52:40 +0000 | [diff] [blame] | 146 | public: |
Andrew Trick | 10ffc2b | 2010-12-24 05:03:26 +0000 | [diff] [blame] | 147 | SchedulePostRATDList( |
Alexey Samsonov | ea0aee6 | 2014-08-20 20:57:26 +0000 | [diff] [blame] | 148 | MachineFunction &MF, MachineLoopInfo &MLI, AliasAnalysis *AA, |
| 149 | const RegisterClassInfo &, |
| 150 | TargetSubtargetInfo::AntiDepBreakMode AntiDepMode, |
| 151 | SmallVectorImpl<const TargetRegisterClass *> &CriticalPathRCs); |
Dan Gohman | ceac7c3 | 2009-01-16 01:33:36 +0000 | [diff] [blame] | 152 | |
Alexander Kornienko | f817c1c | 2015-04-11 02:11:45 +0000 | [diff] [blame] | 153 | ~SchedulePostRATDList() override; |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 154 | |
Andrew Trick | 52226d4 | 2012-03-07 23:00:49 +0000 | [diff] [blame] | 155 | /// startBlock - Initialize register live-range state for scheduling in |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 156 | /// this block. |
| 157 | /// |
Craig Topper | 4584cd5 | 2014-03-07 09:26:03 +0000 | [diff] [blame] | 158 | void startBlock(MachineBasicBlock *BB) override; |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 159 | |
Andrew Trick | a53e101 | 2013-08-23 17:48:33 +0000 | [diff] [blame] | 160 | // Set the index of RegionEnd within the current BB. |
| 161 | void setEndIndex(unsigned EndIdx) { EndIndex = EndIdx; } |
| 162 | |
Andrew Trick | 60cf03e | 2012-03-07 05:21:52 +0000 | [diff] [blame] | 163 | /// Initialize the scheduler state for the next scheduling region. |
Craig Topper | 4584cd5 | 2014-03-07 09:26:03 +0000 | [diff] [blame] | 164 | void enterRegion(MachineBasicBlock *bb, |
| 165 | MachineBasicBlock::iterator begin, |
| 166 | MachineBasicBlock::iterator end, |
| 167 | unsigned regioninstrs) override; |
Andrew Trick | 60cf03e | 2012-03-07 05:21:52 +0000 | [diff] [blame] | 168 | |
| 169 | /// Notify that the scheduler has finished scheduling the current region. |
Craig Topper | 4584cd5 | 2014-03-07 09:26:03 +0000 | [diff] [blame] | 170 | void exitRegion() override; |
Andrew Trick | 60cf03e | 2012-03-07 05:21:52 +0000 | [diff] [blame] | 171 | |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 172 | /// Schedule - Schedule the instruction range using list scheduling. |
| 173 | /// |
Craig Topper | 4584cd5 | 2014-03-07 09:26:03 +0000 | [diff] [blame] | 174 | void schedule() override; |
Jim Grosbach | d772bde | 2010-05-14 21:19:48 +0000 | [diff] [blame] | 175 | |
Andrew Trick | e932bb7 | 2012-03-07 05:21:44 +0000 | [diff] [blame] | 176 | void EmitSchedule(); |
| 177 | |
Dan Gohman | 682a2d1 | 2009-10-21 01:44:44 +0000 | [diff] [blame] | 178 | /// Observe - Update liveness information to account for the current |
| 179 | /// instruction, which will not be scheduled. |
| 180 | /// |
Duncan P. N. Exon Smith | 5e6e8c7 | 2016-02-27 19:33:37 +0000 | [diff] [blame] | 181 | void Observe(MachineInstr &MI, unsigned Count); |
Dan Gohman | 682a2d1 | 2009-10-21 01:44:44 +0000 | [diff] [blame] | 182 | |
Andrew Trick | 52226d4 | 2012-03-07 23:00:49 +0000 | [diff] [blame] | 183 | /// finishBlock - Clean up register live-range state. |
Dan Gohman | 682a2d1 | 2009-10-21 01:44:44 +0000 | [diff] [blame] | 184 | /// |
Craig Topper | 4584cd5 | 2014-03-07 09:26:03 +0000 | [diff] [blame] | 185 | void finishBlock() override; |
Dan Gohman | 682a2d1 | 2009-10-21 01:44:44 +0000 | [diff] [blame] | 186 | |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 187 | private: |
Krzysztof Parzyszek | 5c61d11 | 2016-03-05 15:45:23 +0000 | [diff] [blame] | 188 | /// Apply each ScheduleDAGMutation step in order. |
| 189 | void postprocessDAG(); |
| 190 | |
David Goodwin | 80a03cc | 2009-11-20 19:32:48 +0000 | [diff] [blame] | 191 | void ReleaseSucc(SUnit *SU, SDep *SuccEdge); |
| 192 | void ReleaseSuccessors(SUnit *SU); |
| 193 | void ScheduleNodeTopDown(SUnit *SU, unsigned CurCycle); |
| 194 | void ListScheduleTopDown(); |
Jim Grosbach | d772bde | 2010-05-14 21:19:48 +0000 | [diff] [blame] | 195 | |
Andrew Trick | edee68c | 2012-03-07 05:21:40 +0000 | [diff] [blame] | 196 | void dumpSchedule() const; |
Hal Finkel | 4fd3b1d | 2013-12-11 22:33:43 +0000 | [diff] [blame] | 197 | void emitNoop(unsigned CurCycle); |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 198 | }; |
Alexander Kornienko | f00654e | 2015-06-23 09:49:53 +0000 | [diff] [blame] | 199 | } |
Dale Johannesen | 2182f06 | 2007-07-13 17:13:54 +0000 | [diff] [blame] | 200 | |
Andrew Trick | 1fa5bcb | 2012-02-08 21:23:13 +0000 | [diff] [blame] | 201 | char &llvm::PostRASchedulerID = PostRAScheduler::ID; |
| 202 | |
Matthias Braun | 1527baa | 2017-05-25 21:26:32 +0000 | [diff] [blame] | 203 | INITIALIZE_PASS(PostRAScheduler, DEBUG_TYPE, |
Andrew Trick | 1fa5bcb | 2012-02-08 21:23:13 +0000 | [diff] [blame] | 204 | "Post RA top-down list latency scheduler", false, false) |
| 205 | |
Andrew Trick | 10ffc2b | 2010-12-24 05:03:26 +0000 | [diff] [blame] | 206 | SchedulePostRATDList::SchedulePostRATDList( |
Alexey Samsonov | ea0aee6 | 2014-08-20 20:57:26 +0000 | [diff] [blame] | 207 | MachineFunction &MF, MachineLoopInfo &MLI, AliasAnalysis *AA, |
| 208 | const RegisterClassInfo &RCI, |
| 209 | TargetSubtargetInfo::AntiDepBreakMode AntiDepMode, |
| 210 | SmallVectorImpl<const TargetRegisterClass *> &CriticalPathRCs) |
Matthias Braun | 93563e7 | 2015-11-03 01:53:29 +0000 | [diff] [blame] | 211 | : ScheduleDAGInstrs(MF, &MLI), AA(AA), EndIndex(0) { |
Andrew Trick | 6b104f8 | 2013-12-28 21:56:55 +0000 | [diff] [blame] | 212 | |
Eric Christopher | d913448 | 2014-08-04 21:25:23 +0000 | [diff] [blame] | 213 | const InstrItineraryData *InstrItins = |
Eric Christopher | b66367a | 2014-10-14 07:17:23 +0000 | [diff] [blame] | 214 | MF.getSubtarget().getInstrItineraryData(); |
Andrew Trick | 10ffc2b | 2010-12-24 05:03:26 +0000 | [diff] [blame] | 215 | HazardRec = |
Eric Christopher | b66367a | 2014-10-14 07:17:23 +0000 | [diff] [blame] | 216 | MF.getSubtarget().getInstrInfo()->CreateTargetPostRAHazardRecognizer( |
Eric Christopher | d913448 | 2014-08-04 21:25:23 +0000 | [diff] [blame] | 217 | InstrItins, this); |
Krzysztof Parzyszek | 5c61d11 | 2016-03-05 15:45:23 +0000 | [diff] [blame] | 218 | MF.getSubtarget().getPostRAMutations(Mutations); |
Preston Gurd | 9a09147 | 2012-04-23 21:39:35 +0000 | [diff] [blame] | 219 | |
| 220 | assert((AntiDepMode == TargetSubtargetInfo::ANTIDEP_NONE || |
| 221 | MRI.tracksLiveness()) && |
| 222 | "Live-ins must be accurate for anti-dependency breaking"); |
Andrew Trick | 10ffc2b | 2010-12-24 05:03:26 +0000 | [diff] [blame] | 223 | AntiDepBreak = |
Evan Cheng | 0d639a2 | 2011-07-01 21:01:15 +0000 | [diff] [blame] | 224 | ((AntiDepMode == TargetSubtargetInfo::ANTIDEP_ALL) ? |
Jakob Stoklund Olesen | 4f5f84c | 2011-06-16 21:56:21 +0000 | [diff] [blame] | 225 | (AntiDepBreaker *)new AggressiveAntiDepBreaker(MF, RCI, CriticalPathRCs) : |
Evan Cheng | 0d639a2 | 2011-07-01 21:01:15 +0000 | [diff] [blame] | 226 | ((AntiDepMode == TargetSubtargetInfo::ANTIDEP_CRITICAL) ? |
Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 227 | (AntiDepBreaker *)new CriticalAntiDepBreaker(MF, RCI) : nullptr)); |
Andrew Trick | 10ffc2b | 2010-12-24 05:03:26 +0000 | [diff] [blame] | 228 | } |
| 229 | |
| 230 | SchedulePostRATDList::~SchedulePostRATDList() { |
| 231 | delete HazardRec; |
| 232 | delete AntiDepBreak; |
| 233 | } |
| 234 | |
Andrew Trick | 60cf03e | 2012-03-07 05:21:52 +0000 | [diff] [blame] | 235 | /// Initialize state associated with the next scheduling region. |
| 236 | void SchedulePostRATDList::enterRegion(MachineBasicBlock *bb, |
| 237 | MachineBasicBlock::iterator begin, |
| 238 | MachineBasicBlock::iterator end, |
Andrew Trick | a53e101 | 2013-08-23 17:48:33 +0000 | [diff] [blame] | 239 | unsigned regioninstrs) { |
| 240 | ScheduleDAGInstrs::enterRegion(bb, begin, end, regioninstrs); |
Andrew Trick | 60cf03e | 2012-03-07 05:21:52 +0000 | [diff] [blame] | 241 | Sequence.clear(); |
| 242 | } |
| 243 | |
| 244 | /// Print the schedule before exiting the region. |
| 245 | void SchedulePostRATDList::exitRegion() { |
| 246 | DEBUG({ |
| 247 | dbgs() << "*** Final schedule ***\n"; |
| 248 | dumpSchedule(); |
| 249 | dbgs() << '\n'; |
| 250 | }); |
| 251 | ScheduleDAGInstrs::exitRegion(); |
| 252 | } |
| 253 | |
Aaron Ballman | 615eb47 | 2017-10-15 14:32:27 +0000 | [diff] [blame^] | 254 | #if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP) |
Andrew Trick | edee68c | 2012-03-07 05:21:40 +0000 | [diff] [blame] | 255 | /// dumpSchedule - dump the scheduled Sequence. |
Matthias Braun | 8c209aa | 2017-01-28 02:02:38 +0000 | [diff] [blame] | 256 | LLVM_DUMP_METHOD void SchedulePostRATDList::dumpSchedule() const { |
Andrew Trick | edee68c | 2012-03-07 05:21:40 +0000 | [diff] [blame] | 257 | for (unsigned i = 0, e = Sequence.size(); i != e; i++) { |
| 258 | if (SUnit *SU = Sequence[i]) |
| 259 | SU->dump(this); |
| 260 | else |
| 261 | dbgs() << "**** NOOP ****\n"; |
| 262 | } |
| 263 | } |
Manman Ren | 742534c | 2012-09-06 19:06:06 +0000 | [diff] [blame] | 264 | #endif |
Andrew Trick | edee68c | 2012-03-07 05:21:40 +0000 | [diff] [blame] | 265 | |
Sanjay Patel | a2f658d | 2014-07-15 22:39:58 +0000 | [diff] [blame] | 266 | bool PostRAScheduler::enablePostRAScheduler( |
| 267 | const TargetSubtargetInfo &ST, |
| 268 | CodeGenOpt::Level OptLevel, |
| 269 | TargetSubtargetInfo::AntiDepBreakMode &Mode, |
| 270 | TargetSubtargetInfo::RegClassVector &CriticalPathRCs) const { |
| 271 | Mode = ST.getAntiDepBreakMode(); |
| 272 | ST.getCriticalPathRCs(CriticalPathRCs); |
Mitch Bodart | 6453501 | 2016-05-19 16:40:49 +0000 | [diff] [blame] | 273 | |
| 274 | // Check for explicit enable/disable of post-ra scheduling. |
| 275 | if (EnablePostRAScheduler.getPosition() > 0) |
| 276 | return EnablePostRAScheduler; |
| 277 | |
Matthias Braun | 39a2afc | 2015-06-13 03:42:16 +0000 | [diff] [blame] | 278 | return ST.enablePostRAScheduler() && |
Sanjay Patel | a2f658d | 2014-07-15 22:39:58 +0000 | [diff] [blame] | 279 | OptLevel >= ST.getOptLevelToEnablePostRAScheduler(); |
| 280 | } |
| 281 | |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 282 | bool PostRAScheduler::runOnMachineFunction(MachineFunction &Fn) { |
Andrew Kaylor | aa641a5 | 2016-04-22 22:06:11 +0000 | [diff] [blame] | 283 | if (skipFunction(*Fn.getFunction())) |
Paul Robinson | 7c99ec5 | 2014-03-31 17:43:35 +0000 | [diff] [blame] | 284 | return false; |
| 285 | |
Eric Christopher | fc6de42 | 2014-08-05 02:39:49 +0000 | [diff] [blame] | 286 | TII = Fn.getSubtarget().getInstrInfo(); |
Andrew Trick | 10ffc2b | 2010-12-24 05:03:26 +0000 | [diff] [blame] | 287 | MachineLoopInfo &MLI = getAnalysis<MachineLoopInfo>(); |
Chandler Carruth | 7b560d4 | 2015-09-09 17:55:00 +0000 | [diff] [blame] | 288 | AliasAnalysis *AA = &getAnalysis<AAResultsWrapperPass>().getAAResults(); |
Andrew Trick | df7e376 | 2012-02-08 21:22:53 +0000 | [diff] [blame] | 289 | TargetPassConfig *PassConfig = &getAnalysis<TargetPassConfig>(); |
| 290 | |
Jakob Stoklund Olesen | 4f5f84c | 2011-06-16 21:56:21 +0000 | [diff] [blame] | 291 | RegClassInfo.runOnMachineFunction(Fn); |
Dan Gohman | 26e9b89 | 2009-10-10 00:15:38 +0000 | [diff] [blame] | 292 | |
Evan Cheng | 7fae11b | 2011-12-14 02:11:42 +0000 | [diff] [blame] | 293 | TargetSubtargetInfo::AntiDepBreakMode AntiDepMode = |
| 294 | TargetSubtargetInfo::ANTIDEP_NONE; |
Craig Topper | 760b134 | 2012-02-22 05:59:10 +0000 | [diff] [blame] | 295 | SmallVector<const TargetRegisterClass*, 4> CriticalPathRCs; |
Mitch Bodart | 6453501 | 2016-05-19 16:40:49 +0000 | [diff] [blame] | 296 | |
| 297 | // Check that post-RA scheduling is enabled for this target. |
| 298 | // This may upgrade the AntiDepMode. |
| 299 | if (!enablePostRAScheduler(Fn.getSubtarget(), PassConfig->getOptLevel(), |
| 300 | AntiDepMode, CriticalPathRCs)) |
| 301 | return false; |
David Goodwin | 17199b5 | 2009-09-30 00:10:16 +0000 | [diff] [blame] | 302 | |
David Goodwin | 02ad4cb | 2009-10-22 23:19:17 +0000 | [diff] [blame] | 303 | // Check for antidep breaking override... |
| 304 | if (EnableAntiDepBreaking.getPosition() > 0) { |
Evan Cheng | 0d639a2 | 2011-07-01 21:01:15 +0000 | [diff] [blame] | 305 | AntiDepMode = (EnableAntiDepBreaking == "all") |
| 306 | ? TargetSubtargetInfo::ANTIDEP_ALL |
| 307 | : ((EnableAntiDepBreaking == "critical") |
| 308 | ? TargetSubtargetInfo::ANTIDEP_CRITICAL |
| 309 | : TargetSubtargetInfo::ANTIDEP_NONE); |
David Goodwin | 02ad4cb | 2009-10-22 23:19:17 +0000 | [diff] [blame] | 310 | } |
| 311 | |
David Greene | aa8ce38 | 2010-01-05 01:26:01 +0000 | [diff] [blame] | 312 | DEBUG(dbgs() << "PostRAScheduler\n"); |
Dale Johannesen | 2182f06 | 2007-07-13 17:13:54 +0000 | [diff] [blame] | 313 | |
Alexey Samsonov | ea0aee6 | 2014-08-20 20:57:26 +0000 | [diff] [blame] | 314 | SchedulePostRATDList Scheduler(Fn, MLI, AA, RegClassInfo, AntiDepMode, |
Andrew Trick | 10ffc2b | 2010-12-24 05:03:26 +0000 | [diff] [blame] | 315 | CriticalPathRCs); |
Dan Gohman | 619ef48 | 2009-01-15 19:20:50 +0000 | [diff] [blame] | 316 | |
Dale Johannesen | 2182f06 | 2007-07-13 17:13:54 +0000 | [diff] [blame] | 317 | // Loop over all of the basic blocks |
Duncan P. N. Exon Smith | 1ff4098 | 2015-10-09 21:05:00 +0000 | [diff] [blame] | 318 | for (auto &MBB : Fn) { |
David Goodwin | 7f65169 | 2009-09-01 18:34:03 +0000 | [diff] [blame] | 319 | #ifndef NDEBUG |
| 320 | // If DebugDiv > 0 then only schedule MBB with (ID % DebugDiv) == DebugMod |
| 321 | if (DebugDiv > 0) { |
| 322 | static int bbcnt = 0; |
| 323 | if (bbcnt++ % DebugDiv != DebugMod) |
| 324 | continue; |
Craig Topper | a538d83 | 2012-08-22 06:07:19 +0000 | [diff] [blame] | 325 | dbgs() << "*** DEBUG scheduling " << Fn.getName() |
Duncan P. N. Exon Smith | 1ff4098 | 2015-10-09 21:05:00 +0000 | [diff] [blame] | 326 | << ":BB#" << MBB.getNumber() << " ***\n"; |
David Goodwin | 7f65169 | 2009-09-01 18:34:03 +0000 | [diff] [blame] | 327 | } |
| 328 | #endif |
| 329 | |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 330 | // Initialize register live-range state for scheduling in this block. |
Duncan P. N. Exon Smith | 1ff4098 | 2015-10-09 21:05:00 +0000 | [diff] [blame] | 331 | Scheduler.startBlock(&MBB); |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 332 | |
Dan Gohman | 5f8a259 | 2009-01-16 22:10:20 +0000 | [diff] [blame] | 333 | // Schedule each sequence of instructions not interrupted by a label |
| 334 | // or anything else that effectively needs to shut down scheduling. |
Duncan P. N. Exon Smith | 1ff4098 | 2015-10-09 21:05:00 +0000 | [diff] [blame] | 335 | MachineBasicBlock::iterator Current = MBB.end(); |
| 336 | unsigned Count = MBB.size(), CurrentCount = Count; |
| 337 | for (MachineBasicBlock::iterator I = Current; I != MBB.begin();) { |
Duncan P. N. Exon Smith | 762c5ca | 2016-07-01 01:18:53 +0000 | [diff] [blame] | 338 | MachineInstr &MI = *std::prev(I); |
Andrew Trick | a53e101 | 2013-08-23 17:48:33 +0000 | [diff] [blame] | 339 | --Count; |
Jakob Stoklund Olesen | a793a59 | 2012-02-23 17:54:21 +0000 | [diff] [blame] | 340 | // Calls are not scheduling boundaries before register allocation, but |
| 341 | // post-ra we don't gain anything by scheduling across calls since we |
| 342 | // don't need to worry about register pressure. |
Duncan P. N. Exon Smith | 762c5ca | 2016-07-01 01:18:53 +0000 | [diff] [blame] | 343 | if (MI.isCall() || TII->isSchedulingBoundary(MI, &MBB, Fn)) { |
Duncan P. N. Exon Smith | 1ff4098 | 2015-10-09 21:05:00 +0000 | [diff] [blame] | 344 | Scheduler.enterRegion(&MBB, I, Current, CurrentCount - Count); |
Andrew Trick | a53e101 | 2013-08-23 17:48:33 +0000 | [diff] [blame] | 345 | Scheduler.setEndIndex(CurrentCount); |
Andrew Trick | 52226d4 | 2012-03-07 23:00:49 +0000 | [diff] [blame] | 346 | Scheduler.schedule(); |
Andrew Trick | 60cf03e | 2012-03-07 05:21:52 +0000 | [diff] [blame] | 347 | Scheduler.exitRegion(); |
Dan Gohman | 25c1653 | 2010-05-01 00:01:06 +0000 | [diff] [blame] | 348 | Scheduler.EmitSchedule(); |
Duncan P. N. Exon Smith | 762c5ca | 2016-07-01 01:18:53 +0000 | [diff] [blame] | 349 | Current = &MI; |
Andrew Trick | a53e101 | 2013-08-23 17:48:33 +0000 | [diff] [blame] | 350 | CurrentCount = Count; |
Duncan P. N. Exon Smith | 762c5ca | 2016-07-01 01:18:53 +0000 | [diff] [blame] | 351 | Scheduler.Observe(MI, CurrentCount); |
Dan Gohman | 5f8a259 | 2009-01-16 22:10:20 +0000 | [diff] [blame] | 352 | } |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 353 | I = MI; |
Duncan P. N. Exon Smith | 762c5ca | 2016-07-01 01:18:53 +0000 | [diff] [blame] | 354 | if (MI.isBundle()) |
| 355 | Count -= MI.getBundleSize(); |
Dan Gohman | d564353 | 2009-02-03 18:57:45 +0000 | [diff] [blame] | 356 | } |
Dan Gohman | dfaf646 | 2009-02-11 04:27:20 +0000 | [diff] [blame] | 357 | assert(Count == 0 && "Instruction count mismatch!"); |
Duncan P. N. Exon Smith | 1ff4098 | 2015-10-09 21:05:00 +0000 | [diff] [blame] | 358 | assert((MBB.begin() == Current || CurrentCount != 0) && |
Dan Gohman | 64613ac | 2009-03-10 18:10:43 +0000 | [diff] [blame] | 359 | "Instruction count mismatch!"); |
Duncan P. N. Exon Smith | 1ff4098 | 2015-10-09 21:05:00 +0000 | [diff] [blame] | 360 | Scheduler.enterRegion(&MBB, MBB.begin(), Current, CurrentCount); |
Andrew Trick | a53e101 | 2013-08-23 17:48:33 +0000 | [diff] [blame] | 361 | Scheduler.setEndIndex(CurrentCount); |
Andrew Trick | 52226d4 | 2012-03-07 23:00:49 +0000 | [diff] [blame] | 362 | Scheduler.schedule(); |
Andrew Trick | 60cf03e | 2012-03-07 05:21:52 +0000 | [diff] [blame] | 363 | Scheduler.exitRegion(); |
Dan Gohman | 25c1653 | 2010-05-01 00:01:06 +0000 | [diff] [blame] | 364 | Scheduler.EmitSchedule(); |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 365 | |
| 366 | // Clean up register live-range state. |
Andrew Trick | 52226d4 | 2012-03-07 23:00:49 +0000 | [diff] [blame] | 367 | Scheduler.finishBlock(); |
David Goodwin | ae6bc82 | 2009-08-25 17:03:05 +0000 | [diff] [blame] | 368 | |
David Goodwin | 6c08cfc | 2009-09-03 22:15:25 +0000 | [diff] [blame] | 369 | // Update register kills |
Matthias Braun | 868bbd4 | 2017-05-27 02:50:50 +0000 | [diff] [blame] | 370 | Scheduler.fixupKills(MBB); |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 371 | } |
Dale Johannesen | 2182f06 | 2007-07-13 17:13:54 +0000 | [diff] [blame] | 372 | |
| 373 | return true; |
| 374 | } |
Jim Grosbach | d772bde | 2010-05-14 21:19:48 +0000 | [diff] [blame] | 375 | |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 376 | /// StartBlock - Initialize register live-range state for scheduling in |
| 377 | /// this block. |
Dan Gohman | ad2134d | 2008-11-25 00:52:40 +0000 | [diff] [blame] | 378 | /// |
Andrew Trick | 52226d4 | 2012-03-07 23:00:49 +0000 | [diff] [blame] | 379 | void SchedulePostRATDList::startBlock(MachineBasicBlock *BB) { |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 380 | // Call the superclass. |
Andrew Trick | 52226d4 | 2012-03-07 23:00:49 +0000 | [diff] [blame] | 381 | ScheduleDAGInstrs::startBlock(BB); |
Dan Gohman | ad2134d | 2008-11-25 00:52:40 +0000 | [diff] [blame] | 382 | |
David Goodwin | 8370485 | 2009-10-26 16:59:04 +0000 | [diff] [blame] | 383 | // Reset the hazard recognizer and anti-dep breaker. |
David Goodwin | 6021b4d | 2009-08-10 15:55:25 +0000 | [diff] [blame] | 384 | HazardRec->Reset(); |
Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 385 | if (AntiDepBreak) |
David Goodwin | 8370485 | 2009-10-26 16:59:04 +0000 | [diff] [blame] | 386 | AntiDepBreak->StartBlock(BB); |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 387 | } |
| 388 | |
| 389 | /// Schedule - Schedule the instruction range using list scheduling. |
| 390 | /// |
Andrew Trick | 52226d4 | 2012-03-07 23:00:49 +0000 | [diff] [blame] | 391 | void SchedulePostRATDList::schedule() { |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 392 | // Build the scheduling graph. |
Andrew Trick | 52226d4 | 2012-03-07 23:00:49 +0000 | [diff] [blame] | 393 | buildSchedGraph(AA); |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 394 | |
Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 395 | if (AntiDepBreak) { |
Jim Grosbach | d772bde | 2010-05-14 21:19:48 +0000 | [diff] [blame] | 396 | unsigned Broken = |
Andrew Trick | 8c207e4 | 2012-03-09 04:29:02 +0000 | [diff] [blame] | 397 | AntiDepBreak->BreakAntiDependencies(SUnits, RegionBegin, RegionEnd, |
| 398 | EndIndex, DbgValues); |
Jim Grosbach | d772bde | 2010-05-14 21:19:48 +0000 | [diff] [blame] | 399 | |
David Goodwin | 80a03cc | 2009-11-20 19:32:48 +0000 | [diff] [blame] | 400 | if (Broken != 0) { |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 401 | // We made changes. Update the dependency graph. |
| 402 | // Theoretically we could update the graph in place: |
| 403 | // When a live range is changed to use a different register, remove |
| 404 | // the def's anti-dependence *and* output-dependence edges due to |
| 405 | // that register, and add new anti-dependence and output-dependence |
| 406 | // edges based on the next live range of the register. |
Andrew Trick | 60cf03e | 2012-03-07 05:21:52 +0000 | [diff] [blame] | 407 | ScheduleDAG::clearDAG(); |
Andrew Trick | 52226d4 | 2012-03-07 23:00:49 +0000 | [diff] [blame] | 408 | buildSchedGraph(AA); |
Jim Grosbach | d772bde | 2010-05-14 21:19:48 +0000 | [diff] [blame] | 409 | |
David Goodwin | 8370485 | 2009-10-26 16:59:04 +0000 | [diff] [blame] | 410 | NumFixedAnti += Broken; |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 411 | } |
| 412 | } |
| 413 | |
Krzysztof Parzyszek | cd99e36 | 2016-03-08 16:54:20 +0000 | [diff] [blame] | 414 | postprocessDAG(); |
| 415 | |
David Greene | aa8ce38 | 2010-01-05 01:26:01 +0000 | [diff] [blame] | 416 | DEBUG(dbgs() << "********** List Scheduling **********\n"); |
Matthias Braun | 9198c67 | 2015-11-06 20:59:02 +0000 | [diff] [blame] | 417 | DEBUG( |
| 418 | for (const SUnit &SU : SUnits) { |
| 419 | SU.dumpAll(this); |
| 420 | dbgs() << '\n'; |
| 421 | } |
| 422 | ); |
David Goodwin | 6021b4d | 2009-08-10 15:55:25 +0000 | [diff] [blame] | 423 | |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 424 | AvailableQueue.initNodes(SUnits); |
David Goodwin | 80a03cc | 2009-11-20 19:32:48 +0000 | [diff] [blame] | 425 | ListScheduleTopDown(); |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 426 | AvailableQueue.releaseState(); |
| 427 | } |
| 428 | |
| 429 | /// Observe - Update liveness information to account for the current |
| 430 | /// instruction, which will not be scheduled. |
| 431 | /// |
Duncan P. N. Exon Smith | 5e6e8c7 | 2016-02-27 19:33:37 +0000 | [diff] [blame] | 432 | void SchedulePostRATDList::Observe(MachineInstr &MI, unsigned Count) { |
Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 433 | if (AntiDepBreak) |
Andrew Trick | a316faa | 2012-03-07 23:00:52 +0000 | [diff] [blame] | 434 | AntiDepBreak->Observe(MI, Count, EndIndex); |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 435 | } |
| 436 | |
| 437 | /// FinishBlock - Clean up register live-range state. |
| 438 | /// |
Andrew Trick | 52226d4 | 2012-03-07 23:00:49 +0000 | [diff] [blame] | 439 | void SchedulePostRATDList::finishBlock() { |
Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 440 | if (AntiDepBreak) |
David Goodwin | 8370485 | 2009-10-26 16:59:04 +0000 | [diff] [blame] | 441 | AntiDepBreak->FinishBlock(); |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 442 | |
| 443 | // Call the superclass. |
Andrew Trick | 52226d4 | 2012-03-07 23:00:49 +0000 | [diff] [blame] | 444 | ScheduleDAGInstrs::finishBlock(); |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 445 | } |
| 446 | |
Krzysztof Parzyszek | 5c61d11 | 2016-03-05 15:45:23 +0000 | [diff] [blame] | 447 | /// Apply each ScheduleDAGMutation step in order. |
| 448 | void SchedulePostRATDList::postprocessDAG() { |
| 449 | for (auto &M : Mutations) |
| 450 | M->apply(this); |
| 451 | } |
| 452 | |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 453 | //===----------------------------------------------------------------------===// |
| 454 | // Top-Down Scheduling |
| 455 | //===----------------------------------------------------------------------===// |
| 456 | |
| 457 | /// ReleaseSucc - Decrement the NumPredsLeft count of a successor. Add it to |
Andrew Trick | f1ff84c | 2012-11-12 19:28:57 +0000 | [diff] [blame] | 458 | /// the PendingQueue if the count reaches zero. |
David Goodwin | 80a03cc | 2009-11-20 19:32:48 +0000 | [diff] [blame] | 459 | void SchedulePostRATDList::ReleaseSucc(SUnit *SU, SDep *SuccEdge) { |
Dan Gohman | 2d17089 | 2008-12-09 22:54:47 +0000 | [diff] [blame] | 460 | SUnit *SuccSU = SuccEdge->getSUnit(); |
Reid Kleckner | 8ff5c19 | 2009-09-30 20:15:38 +0000 | [diff] [blame] | 461 | |
Andrew Trick | 4b1f9e3 | 2012-11-13 02:35:06 +0000 | [diff] [blame] | 462 | if (SuccEdge->isWeak()) { |
Andrew Trick | f1ff84c | 2012-11-12 19:28:57 +0000 | [diff] [blame] | 463 | --SuccSU->WeakPredsLeft; |
| 464 | return; |
| 465 | } |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 466 | #ifndef NDEBUG |
Reid Kleckner | 8ff5c19 | 2009-09-30 20:15:38 +0000 | [diff] [blame] | 467 | if (SuccSU->NumPredsLeft == 0) { |
David Greene | aa8ce38 | 2010-01-05 01:26:01 +0000 | [diff] [blame] | 468 | dbgs() << "*** Scheduling failed! ***\n"; |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 469 | SuccSU->dump(this); |
David Greene | aa8ce38 | 2010-01-05 01:26:01 +0000 | [diff] [blame] | 470 | dbgs() << " has been released too many times!\n"; |
Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 471 | llvm_unreachable(nullptr); |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 472 | } |
| 473 | #endif |
Reid Kleckner | 8ff5c19 | 2009-09-30 20:15:38 +0000 | [diff] [blame] | 474 | --SuccSU->NumPredsLeft; |
| 475 | |
Andrew Trick | 84f9ad9 | 2011-05-06 18:14:32 +0000 | [diff] [blame] | 476 | // Standard scheduler algorithms will recompute the depth of the successor |
Andrew Trick | aab77fe | 2011-05-06 17:09:08 +0000 | [diff] [blame] | 477 | // here as such: |
| 478 | // SuccSU->setDepthToAtLeast(SU->getDepth() + SuccEdge->getLatency()); |
| 479 | // |
| 480 | // However, we lazily compute node depth instead. Note that |
| 481 | // ScheduleNodeTopDown has already updated the depth of this node which causes |
| 482 | // all descendents to be marked dirty. Setting the successor depth explicitly |
| 483 | // here would cause depth to be recomputed for all its ancestors. If the |
| 484 | // successor is not yet ready (because of a transitively redundant edge) then |
| 485 | // this causes depth computation to be quadratic in the size of the DAG. |
Jim Grosbach | d772bde | 2010-05-14 21:19:48 +0000 | [diff] [blame] | 486 | |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 487 | // If all the node's predecessors are scheduled, this node is ready |
| 488 | // to be scheduled. Ignore the special ExitSU node. |
| 489 | if (SuccSU->NumPredsLeft == 0 && SuccSU != &ExitSU) |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 490 | PendingQueue.push_back(SuccSU); |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 491 | } |
| 492 | |
| 493 | /// ReleaseSuccessors - Call ReleaseSucc on each of SU's successors. |
David Goodwin | 80a03cc | 2009-11-20 19:32:48 +0000 | [diff] [blame] | 494 | void SchedulePostRATDList::ReleaseSuccessors(SUnit *SU) { |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 495 | for (SUnit::succ_iterator I = SU->Succs.begin(), E = SU->Succs.end(); |
David Goodwin | 8501dbbe | 2009-11-03 20:57:50 +0000 | [diff] [blame] | 496 | I != E; ++I) { |
David Goodwin | 80a03cc | 2009-11-20 19:32:48 +0000 | [diff] [blame] | 497 | ReleaseSucc(SU, &*I); |
David Goodwin | 8501dbbe | 2009-11-03 20:57:50 +0000 | [diff] [blame] | 498 | } |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 499 | } |
| 500 | |
| 501 | /// ScheduleNodeTopDown - Add the node to the schedule. Decrement the pending |
| 502 | /// count of its successors. If a successor pending count is zero, add it to |
| 503 | /// the Available queue. |
David Goodwin | 80a03cc | 2009-11-20 19:32:48 +0000 | [diff] [blame] | 504 | void SchedulePostRATDList::ScheduleNodeTopDown(SUnit *SU, unsigned CurCycle) { |
David Greene | aa8ce38 | 2010-01-05 01:26:01 +0000 | [diff] [blame] | 505 | DEBUG(dbgs() << "*** Scheduling [" << CurCycle << "]: "); |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 506 | DEBUG(SU->dump(this)); |
Jim Grosbach | d772bde | 2010-05-14 21:19:48 +0000 | [diff] [blame] | 507 | |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 508 | Sequence.push_back(SU); |
Jim Grosbach | d772bde | 2010-05-14 21:19:48 +0000 | [diff] [blame] | 509 | assert(CurCycle >= SU->getDepth() && |
David Goodwin | 8501dbbe | 2009-11-03 20:57:50 +0000 | [diff] [blame] | 510 | "Node scheduled above its depth!"); |
David Goodwin | 80a03cc | 2009-11-20 19:32:48 +0000 | [diff] [blame] | 511 | SU->setDepthToAtLeast(CurCycle); |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 512 | |
David Goodwin | 80a03cc | 2009-11-20 19:32:48 +0000 | [diff] [blame] | 513 | ReleaseSuccessors(SU); |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 514 | SU->isScheduled = true; |
Andrew Trick | 52226d4 | 2012-03-07 23:00:49 +0000 | [diff] [blame] | 515 | AvailableQueue.scheduledNode(SU); |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 516 | } |
| 517 | |
Hal Finkel | 4fd3b1d | 2013-12-11 22:33:43 +0000 | [diff] [blame] | 518 | /// emitNoop - Add a noop to the current instruction sequence. |
| 519 | void SchedulePostRATDList::emitNoop(unsigned CurCycle) { |
| 520 | DEBUG(dbgs() << "*** Emitting noop in cycle " << CurCycle << '\n'); |
| 521 | HazardRec->EmitNoop(); |
Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 522 | Sequence.push_back(nullptr); // NULL here means noop |
Hal Finkel | 4fd3b1d | 2013-12-11 22:33:43 +0000 | [diff] [blame] | 523 | ++NumNoops; |
| 524 | } |
| 525 | |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 526 | /// ListScheduleTopDown - The main loop of list scheduling for top-down |
| 527 | /// schedulers. |
David Goodwin | 80a03cc | 2009-11-20 19:32:48 +0000 | [diff] [blame] | 528 | void SchedulePostRATDList::ListScheduleTopDown() { |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 529 | unsigned CurCycle = 0; |
Jim Grosbach | d772bde | 2010-05-14 21:19:48 +0000 | [diff] [blame] | 530 | |
David Goodwin | 8501dbbe | 2009-11-03 20:57:50 +0000 | [diff] [blame] | 531 | // We're scheduling top-down but we're visiting the regions in |
| 532 | // bottom-up order, so we don't know the hazards at the start of a |
| 533 | // region. So assume no hazards (this should usually be ok as most |
| 534 | // blocks are a single region). |
| 535 | HazardRec->Reset(); |
| 536 | |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 537 | // Release any successors of the special Entry node. |
David Goodwin | 80a03cc | 2009-11-20 19:32:48 +0000 | [diff] [blame] | 538 | ReleaseSuccessors(&EntrySU); |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 539 | |
David Goodwin | 80a03cc | 2009-11-20 19:32:48 +0000 | [diff] [blame] | 540 | // Add all leaves to Available queue. |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 541 | for (unsigned i = 0, e = SUnits.size(); i != e; ++i) { |
| 542 | // It is available if it has no predecessors. |
Andrew Trick | f1ff84c | 2012-11-12 19:28:57 +0000 | [diff] [blame] | 543 | if (!SUnits[i].NumPredsLeft && !SUnits[i].isAvailable) { |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 544 | AvailableQueue.push(&SUnits[i]); |
| 545 | SUnits[i].isAvailable = true; |
| 546 | } |
| 547 | } |
Dan Gohman | b954343 | 2009-02-10 23:27:53 +0000 | [diff] [blame] | 548 | |
David Goodwin | 1f8c7a7 | 2009-08-12 21:47:46 +0000 | [diff] [blame] | 549 | // In any cycle where we can't schedule any instructions, we must |
| 550 | // stall or emit a noop, depending on the target. |
Benjamin Kramer | e3c9d23 | 2009-09-06 12:10:17 +0000 | [diff] [blame] | 551 | bool CycleHasInsts = false; |
David Goodwin | 1f8c7a7 | 2009-08-12 21:47:46 +0000 | [diff] [blame] | 552 | |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 553 | // While Available queue is not empty, grab the node with the highest |
| 554 | // priority. If it is not ready put it back. Schedule the node. |
Dan Gohman | ceac7c3 | 2009-01-16 01:33:36 +0000 | [diff] [blame] | 555 | std::vector<SUnit*> NotReady; |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 556 | Sequence.reserve(SUnits.size()); |
| 557 | while (!AvailableQueue.empty() || !PendingQueue.empty()) { |
| 558 | // Check to see if any of the pending instructions are ready to issue. If |
| 559 | // so, add them to the available queue. |
Dan Gohman | dddc1ac | 2008-12-16 03:25:46 +0000 | [diff] [blame] | 560 | unsigned MinDepth = ~0u; |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 561 | for (unsigned i = 0, e = PendingQueue.size(); i != e; ++i) { |
David Goodwin | 80a03cc | 2009-11-20 19:32:48 +0000 | [diff] [blame] | 562 | if (PendingQueue[i]->getDepth() <= CurCycle) { |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 563 | AvailableQueue.push(PendingQueue[i]); |
| 564 | PendingQueue[i]->isAvailable = true; |
| 565 | PendingQueue[i] = PendingQueue.back(); |
| 566 | PendingQueue.pop_back(); |
| 567 | --i; --e; |
David Goodwin | 80a03cc | 2009-11-20 19:32:48 +0000 | [diff] [blame] | 568 | } else if (PendingQueue[i]->getDepth() < MinDepth) |
| 569 | MinDepth = PendingQueue[i]->getDepth(); |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 570 | } |
David Goodwin | ebd694b | 2009-08-11 17:35:23 +0000 | [diff] [blame] | 571 | |
Andrew Trick | 10ffc2b | 2010-12-24 05:03:26 +0000 | [diff] [blame] | 572 | DEBUG(dbgs() << "\n*** Examining Available\n"; AvailableQueue.dump(this)); |
David Goodwin | ebd694b | 2009-08-11 17:35:23 +0000 | [diff] [blame] | 573 | |
Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 574 | SUnit *FoundSUnit = nullptr, *NotPreferredSUnit = nullptr; |
Dan Gohman | ceac7c3 | 2009-01-16 01:33:36 +0000 | [diff] [blame] | 575 | bool HasNoopHazards = false; |
| 576 | while (!AvailableQueue.empty()) { |
| 577 | SUnit *CurSUnit = AvailableQueue.pop(); |
| 578 | |
| 579 | ScheduleHazardRecognizer::HazardType HT = |
Andrew Trick | 10ffc2b | 2010-12-24 05:03:26 +0000 | [diff] [blame] | 580 | HazardRec->getHazardType(CurSUnit, 0/*no stalls*/); |
Dan Gohman | ceac7c3 | 2009-01-16 01:33:36 +0000 | [diff] [blame] | 581 | if (HT == ScheduleHazardRecognizer::NoHazard) { |
Hal Finkel | 4fd3b1d | 2013-12-11 22:33:43 +0000 | [diff] [blame] | 582 | if (HazardRec->ShouldPreferAnother(CurSUnit)) { |
| 583 | if (!NotPreferredSUnit) { |
NAKAMURA Takumi | f51a34e | 2014-10-29 15:23:11 +0000 | [diff] [blame] | 584 | // If this is the first non-preferred node for this cycle, then |
| 585 | // record it and continue searching for a preferred node. If this |
| 586 | // is not the first non-preferred node, then treat it as though |
| 587 | // there had been a hazard. |
Hal Finkel | 4fd3b1d | 2013-12-11 22:33:43 +0000 | [diff] [blame] | 588 | NotPreferredSUnit = CurSUnit; |
| 589 | continue; |
| 590 | } |
| 591 | } else { |
| 592 | FoundSUnit = CurSUnit; |
| 593 | break; |
| 594 | } |
Dan Gohman | ceac7c3 | 2009-01-16 01:33:36 +0000 | [diff] [blame] | 595 | } |
| 596 | |
| 597 | // Remember if this is a noop hazard. |
| 598 | HasNoopHazards |= HT == ScheduleHazardRecognizer::NoopHazard; |
| 599 | |
| 600 | NotReady.push_back(CurSUnit); |
| 601 | } |
| 602 | |
Hal Finkel | 4fd3b1d | 2013-12-11 22:33:43 +0000 | [diff] [blame] | 603 | // If we have a non-preferred node, push it back onto the available list. |
| 604 | // If we did not find a preferred node, then schedule this first |
| 605 | // non-preferred node. |
| 606 | if (NotPreferredSUnit) { |
| 607 | if (!FoundSUnit) { |
| 608 | DEBUG(dbgs() << "*** Will schedule a non-preferred instruction...\n"); |
| 609 | FoundSUnit = NotPreferredSUnit; |
| 610 | } else { |
| 611 | AvailableQueue.push(NotPreferredSUnit); |
| 612 | } |
| 613 | |
Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 614 | NotPreferredSUnit = nullptr; |
Hal Finkel | 4fd3b1d | 2013-12-11 22:33:43 +0000 | [diff] [blame] | 615 | } |
| 616 | |
Dan Gohman | ceac7c3 | 2009-01-16 01:33:36 +0000 | [diff] [blame] | 617 | // Add the nodes that aren't ready back onto the available list. |
| 618 | if (!NotReady.empty()) { |
| 619 | AvailableQueue.push_all(NotReady); |
| 620 | NotReady.clear(); |
| 621 | } |
| 622 | |
David Goodwin | 8501dbbe | 2009-11-03 20:57:50 +0000 | [diff] [blame] | 623 | // If we found a node to schedule... |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 624 | if (FoundSUnit) { |
Hal Finkel | 4fd3b1d | 2013-12-11 22:33:43 +0000 | [diff] [blame] | 625 | // If we need to emit noops prior to this instruction, then do so. |
| 626 | unsigned NumPreNoops = HazardRec->PreEmitNoops(FoundSUnit); |
| 627 | for (unsigned i = 0; i != NumPreNoops; ++i) |
| 628 | emitNoop(CurCycle); |
| 629 | |
David Goodwin | 8501dbbe | 2009-11-03 20:57:50 +0000 | [diff] [blame] | 630 | // ... schedule the node... |
David Goodwin | 80a03cc | 2009-11-20 19:32:48 +0000 | [diff] [blame] | 631 | ScheduleNodeTopDown(FoundSUnit, CurCycle); |
Dan Gohman | ceac7c3 | 2009-01-16 01:33:36 +0000 | [diff] [blame] | 632 | HazardRec->EmitInstruction(FoundSUnit); |
Benjamin Kramer | e3c9d23 | 2009-09-06 12:10:17 +0000 | [diff] [blame] | 633 | CycleHasInsts = true; |
Andrew Trick | 18c9b37 | 2011-06-01 03:27:56 +0000 | [diff] [blame] | 634 | if (HazardRec->atIssueLimit()) { |
| 635 | DEBUG(dbgs() << "*** Max instructions per cycle " << CurCycle << '\n'); |
| 636 | HazardRec->AdvanceCycle(); |
| 637 | ++CurCycle; |
| 638 | CycleHasInsts = false; |
| 639 | } |
Dan Gohman | ceac7c3 | 2009-01-16 01:33:36 +0000 | [diff] [blame] | 640 | } else { |
Benjamin Kramer | e3c9d23 | 2009-09-06 12:10:17 +0000 | [diff] [blame] | 641 | if (CycleHasInsts) { |
David Greene | aa8ce38 | 2010-01-05 01:26:01 +0000 | [diff] [blame] | 642 | DEBUG(dbgs() << "*** Finished cycle " << CurCycle << '\n'); |
David Goodwin | 1f8c7a7 | 2009-08-12 21:47:46 +0000 | [diff] [blame] | 643 | HazardRec->AdvanceCycle(); |
| 644 | } else if (!HasNoopHazards) { |
| 645 | // Otherwise, we have a pipeline stall, but no other problem, |
| 646 | // just advance the current cycle and try again. |
David Greene | aa8ce38 | 2010-01-05 01:26:01 +0000 | [diff] [blame] | 647 | DEBUG(dbgs() << "*** Stall in cycle " << CurCycle << '\n'); |
David Goodwin | 1f8c7a7 | 2009-08-12 21:47:46 +0000 | [diff] [blame] | 648 | HazardRec->AdvanceCycle(); |
David Goodwin | 80a03cc | 2009-11-20 19:32:48 +0000 | [diff] [blame] | 649 | ++NumStalls; |
David Goodwin | 1f8c7a7 | 2009-08-12 21:47:46 +0000 | [diff] [blame] | 650 | } else { |
| 651 | // Otherwise, we have no instructions to issue and we have instructions |
| 652 | // that will fault if we don't do this right. This is the case for |
| 653 | // processors without pipeline interlocks and other cases. |
Hal Finkel | 4fd3b1d | 2013-12-11 22:33:43 +0000 | [diff] [blame] | 654 | emitNoop(CurCycle); |
David Goodwin | 1f8c7a7 | 2009-08-12 21:47:46 +0000 | [diff] [blame] | 655 | } |
| 656 | |
Dan Gohman | ceac7c3 | 2009-01-16 01:33:36 +0000 | [diff] [blame] | 657 | ++CurCycle; |
Benjamin Kramer | e3c9d23 | 2009-09-06 12:10:17 +0000 | [diff] [blame] | 658 | CycleHasInsts = false; |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 659 | } |
| 660 | } |
| 661 | |
| 662 | #ifndef NDEBUG |
Andrew Trick | 46a5866 | 2012-03-07 05:21:36 +0000 | [diff] [blame] | 663 | unsigned ScheduledNodes = VerifyScheduledDAG(/*isBottomUp=*/false); |
| 664 | unsigned Noops = 0; |
| 665 | for (unsigned i = 0, e = Sequence.size(); i != e; ++i) |
| 666 | if (!Sequence[i]) |
| 667 | ++Noops; |
| 668 | assert(Sequence.size() - Noops == ScheduledNodes && |
| 669 | "The number of nodes scheduled doesn't match the expected number!"); |
| 670 | #endif // NDEBUG |
Dan Gohman | 60cb69e | 2008-11-19 23:18:57 +0000 | [diff] [blame] | 671 | } |
Andrew Trick | e932bb7 | 2012-03-07 05:21:44 +0000 | [diff] [blame] | 672 | |
| 673 | // EmitSchedule - Emit the machine code in scheduled order. |
| 674 | void SchedulePostRATDList::EmitSchedule() { |
Andrew Trick | 8c207e4 | 2012-03-09 04:29:02 +0000 | [diff] [blame] | 675 | RegionBegin = RegionEnd; |
Andrew Trick | e932bb7 | 2012-03-07 05:21:44 +0000 | [diff] [blame] | 676 | |
| 677 | // If first instruction was a DBG_VALUE then put it back. |
| 678 | if (FirstDbgValue) |
Andrew Trick | 8c207e4 | 2012-03-09 04:29:02 +0000 | [diff] [blame] | 679 | BB->splice(RegionEnd, BB, FirstDbgValue); |
Andrew Trick | e932bb7 | 2012-03-07 05:21:44 +0000 | [diff] [blame] | 680 | |
| 681 | // Then re-insert them according to the given schedule. |
| 682 | for (unsigned i = 0, e = Sequence.size(); i != e; i++) { |
| 683 | if (SUnit *SU = Sequence[i]) |
Andrew Trick | 8c207e4 | 2012-03-09 04:29:02 +0000 | [diff] [blame] | 684 | BB->splice(RegionEnd, BB, SU->getInstr()); |
Andrew Trick | e932bb7 | 2012-03-07 05:21:44 +0000 | [diff] [blame] | 685 | else |
| 686 | // Null SUnit* is a noop. |
Andrew Trick | 8c207e4 | 2012-03-09 04:29:02 +0000 | [diff] [blame] | 687 | TII->insertNoop(*BB, RegionEnd); |
Andrew Trick | e932bb7 | 2012-03-07 05:21:44 +0000 | [diff] [blame] | 688 | |
| 689 | // Update the Begin iterator, as the first instruction in the block |
| 690 | // may have been scheduled later. |
| 691 | if (i == 0) |
Benjamin Kramer | b6d0bd4 | 2014-03-02 12:27:27 +0000 | [diff] [blame] | 692 | RegionBegin = std::prev(RegionEnd); |
Andrew Trick | e932bb7 | 2012-03-07 05:21:44 +0000 | [diff] [blame] | 693 | } |
| 694 | |
| 695 | // Reinsert any remaining debug_values. |
| 696 | for (std::vector<std::pair<MachineInstr *, MachineInstr *> >::iterator |
| 697 | DI = DbgValues.end(), DE = DbgValues.begin(); DI != DE; --DI) { |
Benjamin Kramer | b6d0bd4 | 2014-03-02 12:27:27 +0000 | [diff] [blame] | 698 | std::pair<MachineInstr *, MachineInstr *> P = *std::prev(DI); |
Andrew Trick | e932bb7 | 2012-03-07 05:21:44 +0000 | [diff] [blame] | 699 | MachineInstr *DbgValue = P.first; |
| 700 | MachineBasicBlock::iterator OrigPrivMI = P.second; |
| 701 | BB->splice(++OrigPrivMI, BB, DbgValue); |
| 702 | } |
| 703 | DbgValues.clear(); |
Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 704 | FirstDbgValue = nullptr; |
Andrew Trick | e932bb7 | 2012-03-07 05:21:44 +0000 | [diff] [blame] | 705 | } |