Reviewed-by: Tapani Pälli <tapani.pa...@intel.com>

On 09/03/2018 09:22 PM, Jason Ekstrand wrote:
As of 07a2098a708a2, brw_nir_optimize calls nir_remove_dead_variables as
the last optimization.  Doing it again is just pointless.
---
  src/intel/compiler/brw_nir.c | 2 --
  1 file changed, 2 deletions(-)

diff --git a/src/intel/compiler/brw_nir.c b/src/intel/compiler/brw_nir.c
index a906a026042..ce865e2ce71 100644
--- a/src/intel/compiler/brw_nir.c
+++ b/src/intel/compiler/brw_nir.c
@@ -715,8 +715,6 @@ brw_preprocess_nir(const struct brw_compiler *compiler, 
nir_shader *nir)
     /* Get rid of split copies */
     nir = brw_nir_optimize(nir, compiler, is_scalar, false);
- OPT(nir_remove_dead_variables, nir_var_local);
-
     return nir;
  }
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to