[Mesa-dev] [PATCH v2 21/82] glsl: Don't do tree grafting on buffer variables

Samuel Iglesias Gonsálvez siglesias at igalia.com
Wed Jun 17 22:20:34 PDT 2015



On 17/06/15 21:19, Jordan Justen wrote:
> On 2015-06-03 00:01:11, Iago Toral Quiroga wrote:
>> Otherwise we can lose writes into the buffers backing the variables.
>> ---
>>  src/glsl/opt_tree_grafting.cpp | 9 +++++----
>>  1 file changed, 5 insertions(+), 4 deletions(-)
>>
>> diff --git a/src/glsl/opt_tree_grafting.cpp b/src/glsl/opt_tree_grafting.cpp
>> index d47613c..7f2ee6c 100644
>> --- a/src/glsl/opt_tree_grafting.cpp
>> +++ b/src/glsl/opt_tree_grafting.cpp
>> @@ -359,10 +359,11 @@ tree_grafting_basic_block(ir_instruction *bb_first,
>>        if (!lhs_var)
>>          continue;
>>  
>> -      if (lhs_var->data.mode == ir_var_function_out ||
>> -         lhs_var->data.mode == ir_var_function_inout ||
>> -          lhs_var->data.mode == ir_var_shader_out)
>> -        continue;
>> +   if (lhs_var->data.mode == ir_var_function_out ||
>> +       lhs_var->data.mode == ir_var_function_inout ||
>> +       lhs_var->data.mode == ir_var_shader_out ||
>> +       lhs_var->data.mode == ir_var_shader_storage)
>> +      continue;
> 
> This indentation looks wrong. If fixed,
> Reviewed-by: Jordan Justen <jordan.l.justen at intel.com>
> 

Actually the indentation is fine: it uses 3-spaces. The old code was
mixing tabs and spaces.

Thanks,

Sam

>>  
>>        ir_variable_refcount_entry *entry = info->refs->get_variable_entry(lhs_var);
>>  
>> -- 
>> 1.9.1
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
> 


More information about the mesa-dev mailing list