[Mesa-dev] [PATCH] glsl: Use correct mode for split components.

Jordan Justen jordan.l.justen at intel.com
Mon May 23 23:21:19 UTC 2016


Reviewed-by: Jordan Justen <jordan.l.justen at intel.com>

On 2016-04-20 06:42:01, Bas Nieuwenhuizen wrote:
> The mode should stay the same as the original struct. In
> particular, shared should not be changed to temporary.
> 
> Signed-off-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
> ---
>  src/compiler/glsl/opt_structure_splitting.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/compiler/glsl/opt_structure_splitting.cpp b/src/compiler/glsl/opt_structure_splitting.cpp
> index 0d18a2f..f4c129e 100644
> --- a/src/compiler/glsl/opt_structure_splitting.cpp
> +++ b/src/compiler/glsl/opt_structure_splitting.cpp
> @@ -351,7 +351,7 @@ do_structure_splitting(exec_list *instructions)
>          entry->components[i] =
>             new(entry->mem_ctx) ir_variable(type->fields.structure[i].type,
>                                             name,
> -                                           ir_var_temporary);
> +                                           (ir_variable_mode) entry->var->data.mode);
>          entry->var->insert_before(entry->components[i]);
>        }
>  
> -- 
> 2.8.0
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list