Doesn't the old behavior also lower compact arrays even with modes = 0?


On Sat, Oct 28, 2017 at 8:36 PM, Jason Ekstrand <ja...@jlekstrand.net> wrote:
> There's no point in walking the program if 100% if we're never going to
> actually lower anything.
> ---
>  src/compiler/nir/nir_lower_indirect_derefs.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/src/compiler/nir/nir_lower_indirect_derefs.c 
> b/src/compiler/nir/nir_lower_indirect_derefs.c
> index c949224..f1e060c 100644
> --- a/src/compiler/nir/nir_lower_indirect_derefs.c
> +++ b/src/compiler/nir/nir_lower_indirect_derefs.c
> @@ -202,6 +202,9 @@ nir_lower_indirect_derefs(nir_shader *shader, 
> nir_variable_mode modes)
>  {
>     bool progress = false;
>
> +   if (modes == 0)
> +      return false;
> +
>     nir_foreach_function(function, shader) {
>        if (function->impl)
>           progress = lower_indirects_impl(function->impl, modes) || progress;
> --
> 2.5.0.400.gff86faf
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to