This is still unreviewed.

Jason, since you reviewed the other patch I sent related to output
initializers, could you have a look at this one too?

Iago

On Tue, 2018-01-23 at 14:11 +0100, Iago Toral Quiroga wrote:
> The SPIR-V parser splits in/out struct variables and creates
> a separate variable for each first-level member of the struct.
> When the struct variable has an initializer this means that we also
> need to split the initializer.
> ---
>  src/compiler/spirv/vtn_variables.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/src/compiler/spirv/vtn_variables.c
> b/src/compiler/spirv/vtn_variables.c
> index eb306d0c4a..ead68b4784 100644
> --- a/src/compiler/spirv/vtn_variables.c
> +++ b/src/compiler/spirv/vtn_variables.c
> @@ -1837,7 +1837,15 @@ vtn_create_variable(struct vtn_builder *b,
> struct vtn_value *val,
>                 interface_type->members[i]->type;
>              var->members[i]->data.mode = nir_mode;
>              var->members[i]->data.patch = var->patch;
> +
> +            if (initializer) {
> +               assert(i < initializer->num_elements);
> +               var->members[i]->constant_initializer =
> +                  nir_constant_clone(initializer->elements[i], var-
> >members[i]);
> +            }
>           }
> +
> +         initializer = NULL;
>        } else {
>           var->var = rzalloc(b->shader, nir_variable);
>           var->var->name = ralloc_strdup(var->var, val->name);
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to