[llvm] r328138 - [TableGen] Use range-based for loops. NFC

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 21 12:30:32 PDT 2018


Author: ctopper
Date: Wed Mar 21 12:30:31 2018
New Revision: 328138

URL: http://llvm.org/viewvc/llvm-project?rev=328138&view=rev
Log:
[TableGen] Use range-based for loops. NFC

Modified:
    llvm/trunk/utils/TableGen/CodeGenSchedule.cpp

Modified: llvm/trunk/utils/TableGen/CodeGenSchedule.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/TableGen/CodeGenSchedule.cpp?rev=328138&r1=328137&r2=328138&view=diff
==============================================================================
--- llvm/trunk/utils/TableGen/CodeGenSchedule.cpp (original)
+++ llvm/trunk/utils/TableGen/CodeGenSchedule.cpp Wed Mar 21 12:30:31 2018
@@ -804,22 +804,21 @@ void CodeGenSchedModels::createInstRWCla
     // Note that InstDefs may be a smaller list than InstRWDef's "Instrs".
     Record *RWModelDef = InstRWDef->getValueAsDef("SchedModel");
     SmallSet<unsigned, 4> RemappedClassIDs;
-    for (ArrayRef<Record*>::const_iterator
-           II = InstDefs.begin(), IE = InstDefs.end(); II != IE; ++II) {
-      unsigned OldSCIdx = InstrClassMap[*II];
+    for (Record *InstDef : InstDefs) {
+      unsigned OldSCIdx = InstrClassMap[InstDef];
       if (OldSCIdx && RemappedClassIDs.insert(OldSCIdx).second) {
-        for (RecIter RI = SchedClasses[OldSCIdx].InstRWs.begin(),
-               RE = SchedClasses[OldSCIdx].InstRWs.end(); RI != RE; ++RI) {
-          if ((*RI)->getValueAsDef("SchedModel") == RWModelDef) {
-            PrintFatalError(InstRWDef->getLoc(), "Overlapping InstRW def " +
-                          (*II)->getName() + " also matches " +
-                          (*RI)->getValue("Instrs")->getValue()->getAsString());
+        for (Record *OldRWDef : SchedClasses[OldSCIdx].InstRWs) {
+          if (OldRWDef->getValueAsDef("SchedModel") == RWModelDef) {
+            PrintFatalError(OldRWDef->getLoc(), "Overlapping InstRW def " +
+                       InstDef->getName() + " also matches " +
+                       OldRWDef->getValue("Instrs")->getValue()->getAsString());
           }
-          assert(*RI != InstRWDef && "SchedClass has duplicate InstRW def");
-          SC.InstRWs.push_back(*RI);
+          assert(OldRWDef != InstRWDef &&
+                 "SchedClass has duplicate InstRW def");
+          SC.InstRWs.push_back(OldRWDef);
         }
       }
-      InstrClassMap[*II] = SCIdx;
+      InstrClassMap[InstDef] = SCIdx;
     }
     SC.InstRWs.push_back(InstRWDef);
   }




More information about the llvm-commits mailing list