[Mesa-dev] [PATCH v2 22/42] glsl: Don't lower_variable_index_to_cond_assign for shared variables

[Iago Toral]

On Tue, 2015-11-17 at 21:54 -0800, Jordan Justen wrote:
> Signed-off-by: Jordan Justen <jordan.l.justen at intel.com>
> ---
>  src/glsl/lower_variable_index_to_cond_assign.cpp | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/src/glsl/lower_variable_index_to_cond_assign.cpp b/src/glsl/lower_variable_index_to_cond_assign.cpp
> index 1ab3afe..a1ba934 100644
> --- a/src/glsl/lower_variable_index_to_cond_assign.cpp
> +++ b/src/glsl/lower_variable_index_to_cond_assign.cpp
> @@ -378,6 +378,9 @@ public:
>        case ir_var_shader_storage:
>  	 return this->lower_uniforms;
>  
> +      case ir_var_shader_shared:
> +	 return false;

I suppose the right thing to do here is to add a lower_shared_variables
parameter to this and take its value from a compiler option that we set
to false, like we do with the other types, but I guess this is good
enough for now:

Reviewed-by: Iago Toral Quiroga <itoral at igalia.com>

>        case ir_var_function_in:
>        case ir_var_const_in:
>           return this->lower_temps;