diff options
author | Eugene Zelenko <eugene.zelenko@gmail.com> | 2017-09-11 23:00:48 +0000 |
---|---|---|
committer | Eugene Zelenko <eugene.zelenko@gmail.com> | 2017-09-11 23:00:48 +0000 |
commit | 32a40564385ba3b0f96b00e281d854ac2a54b457 (patch) | |
tree | 922185934934ea85195849c9c1bc6293461f0f52 /llvm/lib/CodeGen/MachineScheduler.cpp | |
parent | 7bc1b28291d6fb60cd055c375a1a94fb6ba7c832 (diff) | |
download | bcm5719-llvm-32a40564385ba3b0f96b00e281d854ac2a54b457.tar.gz bcm5719-llvm-32a40564385ba3b0f96b00e281d854ac2a54b457.zip |
[CodeGen] Fix some Clang-tidy modernize-use-using and Include What You Use warnings; other minor fixes (NFC).
llvm-svn: 312971
Diffstat (limited to 'llvm/lib/CodeGen/MachineScheduler.cpp')
-rw-r--r-- | llvm/lib/CodeGen/MachineScheduler.cpp | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/llvm/lib/CodeGen/MachineScheduler.cpp b/llvm/lib/CodeGen/MachineScheduler.cpp index 0257538e230..c64eccd286d 100644 --- a/llvm/lib/CodeGen/MachineScheduler.cpp +++ b/llvm/lib/CodeGen/MachineScheduler.cpp @@ -200,8 +200,7 @@ INITIALIZE_PASS_DEPENDENCY(LiveIntervals) INITIALIZE_PASS_END(MachineScheduler, DEBUG_TYPE, "Machine Instruction Scheduler", false, false) -MachineScheduler::MachineScheduler() -: MachineSchedulerBase(ID) { +MachineScheduler::MachineScheduler() : MachineSchedulerBase(ID) { initializeMachineSchedulerPass(*PassRegistry::getPassRegistry()); } @@ -225,8 +224,7 @@ char &llvm::PostMachineSchedulerID = PostMachineScheduler::ID; INITIALIZE_PASS(PostMachineScheduler, "postmisched", "PostRA Machine Instruction Scheduler", false, false) -PostMachineScheduler::PostMachineScheduler() -: MachineSchedulerBase(ID) { +PostMachineScheduler::PostMachineScheduler() : MachineSchedulerBase(ID) { initializePostMachineSchedulerPass(*PassRegistry::getPassRegistry()); } @@ -448,12 +446,14 @@ struct SchedRegion { MachineBasicBlock::iterator RegionBegin; MachineBasicBlock::iterator RegionEnd; unsigned NumRegionInstrs; + SchedRegion(MachineBasicBlock::iterator B, MachineBasicBlock::iterator E, unsigned N) : RegionBegin(B), RegionEnd(E), NumRegionInstrs(N) {} }; -typedef SmallVector<SchedRegion, 16> MBBRegionsVector; +using MBBRegionsVector = SmallVector<SchedRegion, 16>; + static void getSchedRegions(MachineBasicBlock *MBB, MBBRegionsVector &Regions, @@ -1589,7 +1589,6 @@ void BaseMemOpClusterMutation::clusterNeighboringMemOps( /// \brief Callback from DAG postProcessing to create cluster edges for loads. void BaseMemOpClusterMutation::apply(ScheduleDAGInstrs *DAGInstrs) { - ScheduleDAGMI *DAG = static_cast<ScheduleDAGMI*>(DAGInstrs); // Map DAG NodeNum to store chain ID. @@ -1635,6 +1634,7 @@ namespace { class CopyConstrain : public ScheduleDAGMutation { // Transient state. SlotIndex RegionBeginIdx; + // RegionEndIdx is the slot index of the last non-debug instruction in the // scheduling region. So we may have RegionBeginIdx == RegionEndIdx. SlotIndex RegionEndIdx; @@ -3247,7 +3247,6 @@ void PostGenericScheduler::registerRoots() { /// \param TryCand refers to the next SUnit candidate, otherwise uninitialized. void PostGenericScheduler::tryCandidate(SchedCandidate &Cand, SchedCandidate &TryCand) { - // Initialize the candidate if needed. if (!Cand.isValid()) { TryCand.Reason = NodeOrder; @@ -3486,6 +3485,7 @@ class InstructionShuffler : public MachineSchedStrategy { // instructions to be scheduled first. PriorityQueue<SUnit*, std::vector<SUnit*>, SUnitOrder<false>> TopQ; + // When scheduling bottom-up, use greater-than as the queue priority. PriorityQueue<SUnit*, std::vector<SUnit*>, SUnitOrder<true>> BottomQ; @@ -3602,6 +3602,7 @@ struct DOTGraphTraits<ScheduleDAGMI*> : public DefaultDOTGraphTraits { SS << " I:" << DFS->getNumInstrs(SU); return SS.str(); } + static std::string getNodeDescription(const SUnit *SU, const ScheduleDAG *G) { return G->getGraphNodeLabel(SU); } @@ -3625,7 +3626,6 @@ struct DOTGraphTraits<ScheduleDAGMI*> : public DefaultDOTGraphTraits { /// viewGraph - Pop up a ghostview window with the reachable parts of the DAG /// rendered using 'dot'. -/// void ScheduleDAGMI::viewGraph(const Twine &Name, const Twine &Title) { #ifndef NDEBUG ViewGraph(this, Name, false, Title); |