On 01/04/16 09:46, Kenneth Graunke wrote:
> On Friday, April 1, 2016 9:40:03 AM PDT Alejandro Piñeiro wrote:
>> Needed because not all the built-in variables are marked as system
>> values, so they still have the mode ir_var_auto. Right now it fixes
>> raising the warning when gl_GlobalInvocationID and
>> gl_LocalInvocationIndex are used.
>>
>> v2: use is_gl_identifier instead of filtering for some names (Ilia
>> Mirkin)
>> ---
>>
>> src/compiler/glsl/ast_to_hir.cpp | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/src/compiler/glsl/ast_to_hir.cpp b/src/compiler/glsl/
> ast_to_hir.cpp
>> index a031231..3fe9007 100644
>> --- a/src/compiler/glsl/ast_to_hir.cpp
>> +++ b/src/compiler/glsl/ast_to_hir.cpp
>> @@ -1905,7 +1905,8 @@ ast_expression::do_hir(exec_list *instructions,
>>
>> if ((var->data.mode == ir_var_auto || var->data.mode ==
> ir_var_shader_out)
>> && !this->is_lhs
>> - && result->variable_referenced()->data.assigned != true) {
>> + && result->variable_referenced()->data.assigned != true
>> + && !is_gl_identifier(var->name)) {
>> _mesa_glsl_warning(&loc, state, "`%s' used uninitialized",
>> this->primary_expression.identifier);
>> }
>>
> Looks good, thanks :)
>
> Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
Thanks to the really quick review. Pushed.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
