On Tue, Apr 10, 2018 at 12:52 PM, Caio Marcelo de Oliveira Filho <
caio.olive...@intel.com> wrote:

> The return value was needed to make use of the old nir_foreach_block
> helper, but not needed anymore with the macro version.
>

Nice bit of archaeology there. :-)


> ---
>  src/compiler/nir/nir_lower_vars_to_ssa.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/src/compiler/nir/nir_lower_vars_to_ssa.c
> b/src/compiler/nir/nir_lower_vars_to_ssa.c
> index 0cc65143e7..fe30c69756 100644
> --- a/src/compiler/nir/nir_lower_vars_to_ssa.c
> +++ b/src/compiler/nir/nir_lower_vars_to_ssa.c
> @@ -407,7 +407,7 @@ register_copy_instr(nir_intrinsic_instr *copy_instr,
>  }
>
>  /* Registers all variable uses in the given block. */
> -static bool
> +static void
>  register_variable_uses_block(nir_block *block,
>                               struct lower_variables_state *state)
>  {
> @@ -434,8 +434,6 @@ register_variable_uses_block(nir_block *block,
>           continue;
>        }
>     }
> -
> -   return true;
>

We can probably go even further and either inline this function or else
(and I think I would prefer this one) move the nir_foreach_block into it
and drop the _block suffix from the name.  In any case, this is better

Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net>


>  }
>
>  /* Walks over all of the copy instructions to or from the given deref_node
> --
> 2.17.0
>
> _______________________________________________
> 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