<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Aug 21, 2018 at 6:00 PM Caio Marcelo de Oliveira Filho <<a href="mailto:caio.oliveira@intel.com">caio.oliveira@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> diff --git a/src/intel/compiler/brw_nir.c b/src/intel/compiler/brw_nir.c<br>
> index 96ad77c3906..5e9da9e1ef2 100644<br>
> --- a/src/intel/compiler/brw_nir.c<br>
> +++ b/src/intel/compiler/brw_nir.c<br>
> @@ -542,6 +542,7 @@ brw_nir_optimize(nir_shader *nir, const struct brw_compiler *compiler,<br>
>     do {<br>
>        progress = false;<br>
>        OPT(nir_split_array_vars, nir_var_local);<br>
> +      OPT(nir_shrink_vec_array_vars, nir_var_local);<br>
<br>
I noticed these new passes are being used only for the nir_va_local<br>
case.  Any reason not go ahead and call them with global too?<br></blockquote><div><br></div><div>Because we inline functions and get rid of all global variables before we get here.</div><div><br></div><div>--Jason<br></div></div></div>