ExecutionDomainFix.cpp - use const refs in for-range loops. NFCI.

Avoid unnecessary copies. Reported by clang-tidy.

1 files changed, 5 insertions, 7 deletions

diff --git a/llvm/lib/CodeGen/ExecutionDomainFix.cpp b/llvm/lib/CodeGen/ExecutionDomainFix.cpp
index 2cca05e..9621ad4 100644
--- a/llvm/lib/CodeGen/ExecutionDomainFix.cpp
+++ b/llvm/lib/CodeGen/ExecutionDomainFix.cpp
@@ -380,7 +380,7 @@ void ExecutionDomainFix::visitSoftInstr(MachineInstr *mi, unsigned mask) {
 
   // Finally set all defs and non-collapsed uses to dv. We must iterate through
   // all the operators, including imp-def ones.
-  for (MachineOperand &mo : mi->operands()) {
+  for (const MachineOperand &mo : mi->operands()) {
     if (!mo.isReg())
       continue;
     for (int rx : regIndices(mo.getReg())) {
@@ -454,16 +454,14 @@ bool ExecutionDomainFix::runOnMachineFunction(MachineFunction &mf) {
   // Traverse the basic blocks.
   LoopTraversal Traversal;
   LoopTraversal::TraversalOrder TraversedMBBOrder = Traversal.traverse(mf);
-  for (LoopTraversal::TraversedMBBInfo TraversedMBB : TraversedMBBOrder) {
+  for (const LoopTraversal::TraversedMBBInfo &TraversedMBB : TraversedMBBOrder)
     processBasicBlock(TraversedMBB);
-  }
 
-  for (LiveRegsDVInfo OutLiveRegs : MBBOutRegsInfos) {
-    for (DomainValue *OutLiveReg : OutLiveRegs) {
+  for (const LiveRegsDVInfo &OutLiveRegs : MBBOutRegsInfos)
+    for (DomainValue *OutLiveReg : OutLiveRegs)
       if (OutLiveReg)
         release(OutLiveReg);
-    }
-  }
+
   MBBOutRegsInfos.clear();
   Avail.clear();
   Allocator.DestroyAll();