From 762c5ca3ee1441b4d2e5554846be614d871e2eca Mon Sep 17 00:00:00 2001 From: "Duncan P. N. Exon Smith" Date: Fri, 1 Jul 2016 01:18:53 +0000 Subject: CodeGen: Use MachineInstr& in PostRASchedulerList, NFC Remove another unnecessary iterator to pointer conversion. llvm-svn: 274315 --- llvm/lib/CodeGen/PostRASchedulerList.cpp | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) (limited to 'llvm/lib/CodeGen/PostRASchedulerList.cpp') diff --git a/llvm/lib/CodeGen/PostRASchedulerList.cpp b/llvm/lib/CodeGen/PostRASchedulerList.cpp index c4d20e46ed3..3fce307f3dd 100644 --- a/llvm/lib/CodeGen/PostRASchedulerList.cpp +++ b/llvm/lib/CodeGen/PostRASchedulerList.cpp @@ -335,24 +335,24 @@ bool PostRAScheduler::runOnMachineFunction(MachineFunction &Fn) { MachineBasicBlock::iterator Current = MBB.end(); unsigned Count = MBB.size(), CurrentCount = Count; for (MachineBasicBlock::iterator I = Current; I != MBB.begin();) { - MachineInstr *MI = std::prev(I); + MachineInstr &MI = *std::prev(I); --Count; // Calls are not scheduling boundaries before register allocation, but // post-ra we don't gain anything by scheduling across calls since we // don't need to worry about register pressure. - if (MI->isCall() || TII->isSchedulingBoundary(*MI, &MBB, Fn)) { + if (MI.isCall() || TII->isSchedulingBoundary(MI, &MBB, Fn)) { Scheduler.enterRegion(&MBB, I, Current, CurrentCount - Count); Scheduler.setEndIndex(CurrentCount); Scheduler.schedule(); Scheduler.exitRegion(); Scheduler.EmitSchedule(); - Current = MI; + Current = &MI; CurrentCount = Count; - Scheduler.Observe(*MI, CurrentCount); + Scheduler.Observe(MI, CurrentCount); } I = MI; - if (MI->isBundle()) - Count -= MI->getBundleSize(); + if (MI.isBundle()) + Count -= MI.getBundleSize(); } assert(Count == 0 && "Instruction count mismatch!"); assert((MBB.begin() == Current || CurrentCount != 0) && -- cgit v1.2.3