From c6cc58e7032239529a5ceaab3dab4aa9e7f898e7 Mon Sep 17 00:00:00 2001 From: Benjamin Kramer Date: Sat, 4 Oct 2014 16:55:56 +0000 Subject: Remove unnecessary copying or replace it with moves in a bunch of places. NFC. llvm-svn: 219061 --- llvm/lib/Target/R600/R600ControlFlowFinalizer.cpp | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) (limited to 'llvm/lib/Target/R600/R600ControlFlowFinalizer.cpp') diff --git a/llvm/lib/Target/R600/R600ControlFlowFinalizer.cpp b/llvm/lib/Target/R600/R600ControlFlowFinalizer.cpp index 4433465..edaf278 100644 --- a/llvm/lib/Target/R600/R600ControlFlowFinalizer.cpp +++ b/llvm/lib/Target/R600/R600ControlFlowFinalizer.cpp @@ -459,11 +459,9 @@ private: void CounterPropagateAddr(MachineInstr *MI, unsigned Addr) const { MI->getOperand(0).setImm(Addr + MI->getOperand(0).getImm()); } - void CounterPropagateAddr(std::set MIs, unsigned Addr) - const { - for (std::set::iterator It = MIs.begin(), E = MIs.end(); - It != E; ++It) { - MachineInstr *MI = *It; + void CounterPropagateAddr(const std::set &MIs, + unsigned Addr) const { + for (MachineInstr *MI : MIs) { CounterPropagateAddr(MI, Addr); } } @@ -543,7 +541,7 @@ public: std::pair > Pair(CfCount, std::set()); Pair.second.insert(MIb); - LoopStack.push_back(Pair); + LoopStack.push_back(std::move(Pair)); MI->eraseFromParent(); CfCount++; break; @@ -551,7 +549,7 @@ public: case AMDGPU::ENDLOOP: { CFStack.popLoop(); std::pair > Pair = - LoopStack.back(); + std::move(LoopStack.back()); LoopStack.pop_back(); CounterPropagateAddr(Pair.second, CfCount); BuildMI(MBB, MI, MBB.findDebugLoc(MI), getHWInstrDesc(CF_END_LOOP)) -- cgit v1.1