[Mesa-dev] [PATCH v2 1/3] nir: NULL check lower_copies_to_load_store()

funfunctor at folklore1984.net funfunctor at folklore1984.net
Mon Jul 3 14:00:53 UTC 2017


On 2017-07-03 08:47, Timothy Arceri wrote:
> Allows us to disable array spliting for arrays of arrays without
> regressing tests such as:
> 
> ES31-CTS.functional.shaders.arrays_of_arrays.return.explicit.struct_3x1x3_fragment

Reviewed-by: Edward O'Callaghan <funfunctor at folklore1984.net>

> ---
>  src/compiler/nir/nir_lower_vars_to_ssa.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/compiler/nir/nir_lower_vars_to_ssa.c
> b/src/compiler/nir/nir_lower_vars_to_ssa.c
> index e5a12eb..31f7e7a 100644
> --- a/src/compiler/nir/nir_lower_vars_to_ssa.c
> +++ b/src/compiler/nir/nir_lower_vars_to_ssa.c
> @@ -441,7 +441,7 @@ static bool
>  lower_copies_to_load_store(struct deref_node *node,
>                             struct lower_variables_state *state)
>  {
> -   if (!node->copies)
> +   if (!node || !node->copies)
>        return true;
> 
>     struct set_entry *copy_entry;



More information about the mesa-dev mailing list