[Mesa-dev] [PATCH] swrast: fix active attribs with atifragshader

[Miklós Máté]

On 07/07/2016 07:11 PM, Miklós Máté wrote:
> On 06/26/2016 09:48 PM, Miklós Máté wrote:
>> Only include the ones that can be used by the shader.
>>
>> This fixes texture coordinates, which were completely wrong,
>> because WPOS was included in the list of attribs. It also
>> increases performance noticeably.
>>
>> Signed-off-by: Miklós Máté <mtmkls at gmail.com>
>> ---
>>   src/mesa/swrast/s_context.c | 3 ++-
>>   1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/src/mesa/swrast/s_context.c b/src/mesa/swrast/s_context.c
>> index 0a5fc7e..a63179c 100644
>> --- a/src/mesa/swrast/s_context.c
>> +++ b/src/mesa/swrast/s_context.c
>> @@ -504,7 +504,8 @@ _swrast_update_active_attribs(struct gl_context 
>> *ctx)
>>         attribsMask &= ~VARYING_BIT_POS; /* WPOS is always handled 
>> specially */
>>      }
>>      else if (ctx->ATIFragmentShader._Enabled) {
>> -      attribsMask = ~0;  /* XXX fix me */
>> +      attribsMask = VARYING_BIT_COL0 | VARYING_BIT_COL1 |
>> +                    VARYING_BIT_FOGC | VARYING_BITS_TEX_ANY;
>>      }
>>      else {
>>         /* fixed function */
>
> ping?
>
ping?