[Mesa-dev] [PATCH 06/13] nir: don't count removal of lcssa_phi as progress

Timothy Arceri timothy.arceri at collabora.com
Thu Dec 22 02:26:09 UTC 2016


V2:
 - make the is_lcssa_phi bool const, suggested by Topi.
---
 src/compiler/nir/nir_opt_remove_phis.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/compiler/nir/nir_opt_remove_phis.c b/src/compiler/nir/nir_opt_remove_phis.c
index acaa6e1..99d5d35 100644
--- a/src/compiler/nir/nir_opt_remove_phis.c
+++ b/src/compiler/nir/nir_opt_remove_phis.c
@@ -73,6 +73,7 @@ remove_phis_block(nir_block *block, nir_builder *b)
          break;
 
       nir_phi_instr *phi = nir_instr_as_phi(instr);
+      const bool is_lcssa_phi = phi->is_lcssa_phi;
 
       nir_ssa_def *def = NULL;
       nir_alu_instr *mov = NULL;
@@ -133,7 +134,8 @@ remove_phis_block(nir_block *block, nir_builder *b)
       nir_ssa_def_rewrite_uses(&phi->dest.ssa, nir_src_for_ssa(def));
       nir_instr_remove(instr);
 
-      progress = true;
+      if (!is_lcssa_phi)
+         progress = true;
    }
 
    return progress;
-- 
2.9.3



More information about the mesa-dev mailing list