[Mesa-dev] [PATCH] mesa: fix name returned for XFB varyings

Martin Peres martin.peres at free.fr
Mon Aug 3 03:57:34 PDT 2015


On 03/08/15 13:50, Tapani Pälli wrote:
>
>
> On 08/03/2015 01:48 PM, Martin Peres wrote:
>> On 03/08/15 11:10, Tapani Pälli wrote:
>>> _mesa_get_program_resource_name has logic to append '[0]' in name
>>> if variable is an array, this should be skipped for XFB varyings
>>> that have array index already appended.
>>>
>>> Fixes:
>>>     ES31-CTS.program_interface_query.transform-feedback-types
>>>
>>> Signed-off-by: Tapani Pälli <tapani.palli at intel.com>
>>> ---
>>>   src/mesa/main/shader_query.cpp | 6 ++++++
>>>   1 file changed, 6 insertions(+)
>>>
>>> diff --git a/src/mesa/main/shader_query.cpp
>>> b/src/mesa/main/shader_query.cpp
>>> index c1a7660..a50c348 100644
>>> --- a/src/mesa/main/shader_query.cpp
>>> +++ b/src/mesa/main/shader_query.cpp
>>> @@ -726,6 +726,12 @@ _mesa_get_program_resource_name(struct
>>> gl_shader_program *shProg,
>>>      bool add_index = !(((programInterface == GL_PROGRAM_INPUT) &&
>>>                          res->StageReferences & (1 <<
>>> MESA_SHADER_GEOMETRY)));
>>> +   /* Transform feedback varyings have possible array index already
>>> appended
>>> +    * in their names.
>>> +    */
>>
>> Possibly have the array index already appended? So how do you detect
>> that here?
>
> By 'possible array index' I mean that not all the XFB varyings are 
> arrays. Maybe better text would be just:
>
> "Transform feedback varyings have array index already appended in 
> their names"
>
> If it's not array, then of course it does not have it.

Yes, this comment makes a lot more sense. But I guess having this 
distinction is the problem. How much work would XFB varyings working in 
the same as any other resource when it comes to the name?


More information about the mesa-dev mailing list