[Mesa-dev] [PATCH] r600g: fix handling of outputs as TEX addr sources

[Christian König]

On 08.02.2012 14:52, Alex Deucher wrote:
> 2012/2/6 Christian König<deathsimple at vodafone.de>:
>> Outputs should be treated in the same way as
>> inputs and temporaries here.
>>
>> Signed-off-by: Christian König<deathsimple at vodafone.de>
> Looks good.  Should probably go to the 8.0 stable branch as well?
Not necessarily, since its an optimization and not a real bugfix.

> Reviewed-by: Alex Deucher<alexdeucher at gmail.com>
Thanks, going to commit it soon,
Christian.
>
>> ---
>>   src/gallium/drivers/r600/r600_shader.c |    3 ++-
>>   1 files changed, 2 insertions(+), 1 deletions(-)
>>
>> diff --git a/src/gallium/drivers/r600/r600_shader.c b/src/gallium/drivers/r600/r600_shader.c
>> index 5b145e6..6d05df6 100644
>> --- a/src/gallium/drivers/r600/r600_shader.c
>> +++ b/src/gallium/drivers/r600/r600_shader.c
>> @@ -3252,7 +3252,8 @@ static inline boolean tgsi_tex_src_requires_loading(struct r600_shader_ctx *ctx,
>>   {
>>         struct tgsi_full_instruction *inst =&ctx->parse.FullToken.FullInstruction;
>>         return  (inst->Src[index].Register.File != TGSI_FILE_TEMPORARY&&
>> -               inst->Src[index].Register.File != TGSI_FILE_INPUT) ||
>> +               inst->Src[index].Register.File != TGSI_FILE_INPUT&&
>> +               inst->Src[index].Register.File != TGSI_FILE_OUTPUT) ||
>>                 ctx->src[index].neg || ctx->src[index].abs;
>>   }
>>
>> --
>> 1.7.5.4
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/mesa-dev