nv50/ir/ra: convert some for loops to Range-based for loops
authorKarol Herbst <kherbst@redhat.com>
Wed, 15 Jan 2020 19:37:27 +0000 (20:37 +0100)
committerMarge Bot <eric+marge@anholt.net>
Mon, 22 Jun 2020 12:29:49 +0000 (12:29 +0000)
I will touch them in the next commit

Signed-off-by: Karol Herbst <kherbst@redhat.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5277>

src/gallium/drivers/nouveau/codegen/nv50_ir_ra.cpp

index 4e5b21d917642a6ad2a36f37704dab5b6ff24a8b..0cde9794efc23d27feed106f195de7f2ce42469a 100644 (file)
@@ -954,9 +954,8 @@ GCRA::coalesceValues(Value *dst, Value *src, bool force)
             rep->id, rep->reg.data.id, val->id);
 
    // set join pointer of all values joined with val
-   for (Value::DefIterator def = val->defs.begin(); def != val->defs.end();
-        ++def)
-      (*def)->get()->join = rep;
+   for (ValueDef *def : val->defs)
+      def->get()->join = rep;
    assert(rep->join == rep && val->join == rep);
 
    // add val's definitions to rep and extend the live interval of its RIG node
@@ -1261,10 +1260,8 @@ GCRA::calculateSpillWeights()
 
       if (!val->noSpill) {
          int rc = 0;
-         for (Value::DefIterator it = val->defs.begin();
-              it != val->defs.end();
-              ++it)
-            rc += (*it)->get()->refCount();
+         for (ValueDef *def : val->defs)
+            rc += def->get()->refCount();
 
          nodes[i].weight =
             (float)rc * (float)rc / (float)nodes[i].livei.extent();
@@ -1372,10 +1369,10 @@ GCRA::checkInterference(const RIG_Node *node, Graph::EdgeIterator& ei)
 
    if (vA->compound | vB->compound) {
       // NOTE: this only works for >aligned< register tuples !
-      for (Value::DefCIterator D = vA->defs.begin(); D != vA->defs.end(); ++D) {
-      for (Value::DefCIterator d = vB->defs.begin(); d != vB->defs.end(); ++d) {
-         const LValue *vD = (*D)->get()->asLValue();
-         const LValue *vd = (*d)->get()->asLValue();
+      for (const ValueDef *D : vA->defs) {
+      for (const ValueDef *d : vB->defs) {
+         const LValue *vD = D->get()->asLValue();
+         const LValue *vd = d->get()->asLValue();
 
          if (!vD->livei.overlaps(vd->livei)) {
             INFO_DBG(prog->dbgFlags, REG_ALLOC, "(%%%i) X (%%%i): no overlap\n",