nir/dead_cf: rephrase definition of a dead loop node
authorCaio Marcelo de Oliveira Filho <caio.oliveira@intel.com>
Mon, 19 Mar 2018 23:34:16 +0000 (16:34 -0700)
committerJason Ekstrand <jason.ekstrand@intel.com>
Wed, 21 Mar 2018 16:35:57 +0000 (09:35 -0700)
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
src/compiler/nir/nir_opt_dead_cf.c

index 53c92ecc28695ea4519d8bcf6f043cbf896b3c7b..b15dfdd0c918a77a2ae056d53c6901b71465e1bf 100644 (file)
@@ -171,20 +171,19 @@ def_not_live_out(nir_ssa_def *def, void *state)
 }
 
 /*
- * Test if a loop is dead. A loop is dead if:
+ * Test if a loop is dead. A loop node is dead if:
  *
  * 1) It has no side effects (i.e. intrinsics which could possibly affect the
  * state of the program aside from producing an SSA value, indicated by a lack
  * of NIR_INTRINSIC_CAN_ELIMINATE).
  *
- * 2) It has no phi nodes after it, since those indicate values inside the
- * loop being used after the loop.
+ * 2) It has no phi instructions after it, since those indicate values inside
+ * the node being used after the node.
  *
- * 3) If there are no phi nodes after the loop, then the only way a value
- * defined inside the loop can be used outside the loop is if its definition
- * dominates the block after the loop. If none of the definitions that
- * dominate the loop exit are used outside the loop, then the loop is dead
- * and it can be deleted.
+ * 3) None of the values defined inside the node is used outside the node,
+ * i.e. none of the definitions that dominate the node exit are used outside.
+ *
+ * If those conditions hold, then the node is dead and can be deleted.
  */
 
 static bool