[Piglit] [PATCH 2/3] arb_program_interface_query: Expect array suffixes on array inputs / outputs

[Ian Romanick]

On 05/26/2016 11:59 PM, Tapani Pälli wrote:
> 
> On 05/26/2016 06:16 PM, Martin Peres wrote:
>> On 26/05/16 01:36, Ian Romanick wrote:
>>> From: Ian Romanick <ian.d.romanick at intel.com>
>>>
>>> The ARB_program_interface_query spec says:
>>>
>>>    * For an active variable declared as an array of basic types, a
>>> single
>>>      entry will be generated, with its name string formed by
>>> concatenating
>>>      the name of the array and the string "[0]".
>>>
>>> I can find no language that implies that geometry or tessellation inputs
>>> or tessellation control outputs should be excluded from this rule.
>>> Various dEQP tests also expect [0] be added to geometry shader inputs.
>>>
>>> Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
>>> Cc: Martin Peres <martin.peres at linux.intel.com>
>>> Cc: Dave Airlie <airlied at redhat.com>
>>
>> I remember discussing with Tapani (Cc:ed) about this, but honestly it
>> escaped me and I would trust draw elements more than my memory on this!
> 
> I'm thought we based this on some spec ... but I can't now find it
> anywhere, it could be that in the end we followed Nvidia's
> implementation. Rationale was along the lines of "naming convention is
> different because the inputs of geometry are always arrays".

Yeah, there was a comment along those lines in the linker.

>> In any case, I am all for not making a special case for geometry or
>> tesselation.
>>
>> Reviewed-by: Martin Peres <martin.peres at linux.intel.com>
> 
> // Tapani