[llvm] b4038fb - MachineTraceMetrics: modernize loops (NFC)

Ramkumar Ramachandra via llvm-commits llvm-commits at lists.llvm.org
Thu May 18 04:11:56 PDT 2023


Author: Ramkumar Ramachandra
Date: 2023-05-18T12:11:43+01:00
New Revision: b4038fb72f0ad1d45668db401f494124b8fe79f9

URL: https://github.com/llvm/llvm-project/commit/b4038fb72f0ad1d45668db401f494124b8fe79f9
DIFF: https://github.com/llvm/llvm-project/commit/b4038fb72f0ad1d45668db401f494124b8fe79f9.diff

LOG: MachineTraceMetrics: modernize loops (NFC)

Differential Revision: https://reviews.llvm.org/D150854

Added: 
    

Modified: 
    llvm/lib/CodeGen/MachineTraceMetrics.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/CodeGen/MachineTraceMetrics.cpp b/llvm/lib/CodeGen/MachineTraceMetrics.cpp
index 62a114df7022..2254c84c354c 100644
--- a/llvm/lib/CodeGen/MachineTraceMetrics.cpp
+++ b/llvm/lib/CodeGen/MachineTraceMetrics.cpp
@@ -1099,10 +1099,7 @@ computeInstrHeights(const MachineBasicBlock *MBB) {
     }
 
     // Go through the block backwards.
-    for (MachineBasicBlock::const_iterator BI = MBB->end(), BB = MBB->begin();
-         BI != BB;) {
-      const MachineInstr &MI = *--BI;
-
+    for (const MachineInstr &MI : reverse(*MBB)) {
       // Find the MI height as determined by virtual register uses in the
       // trace below.
       unsigned Cycle = 0;
@@ -1149,11 +1146,10 @@ computeInstrHeights(const MachineBasicBlock *MBB) {
     }
 
     // Transfer the live regunits to the live-in list.
-    for (SparseSet<LiveRegUnit>::const_iterator
-         RI = RegUnits.begin(), RE = RegUnits.end(); RI != RE; ++RI) {
-      TBI.LiveIns.push_back(LiveInReg(RI->RegUnit, RI->Cycle));
-      LLVM_DEBUG(dbgs() << ' ' << printRegUnit(RI->RegUnit, MTM.TRI) << '@'
-                        << RI->Cycle);
+    for (const LiveRegUnit &RU : RegUnits) {
+      TBI.LiveIns.push_back(LiveInReg(RU.RegUnit, RU.Cycle));
+      LLVM_DEBUG(dbgs() << ' ' << printRegUnit(RU.RegUnit, MTM.TRI) << '@'
+                        << RU.Cycle);
     }
     LLVM_DEBUG(dbgs() << '\n');
 


        


More information about the llvm-commits mailing list