[Mesa-dev] [PATCH 1/7] radeonsi: switch to v*i8 for resources and samplers

Christian König deathsimple at vodafone.de
Wed Mar 6 02:47:11 PST 2013


Am 05.03.2013 18:18, schrieb Michel Dänzer:
> On Die, 2013-03-05 at 15:27 +0100, Christian König wrote:
>> From: Christian König <christian.koenig at amd.com>
>>
>> Signed-off-by: Christian König <christian.koenig at amd.com>
>> ---
>>   src/gallium/drivers/radeonsi/radeonsi_shader.c |   31 +++++++++---------------
>>   1 file changed, 12 insertions(+), 19 deletions(-)
>>
>> diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c b/src/gallium/drivers/radeonsi/radeonsi_shader.c
>> index 7922928..958d3a3 100644
>> --- a/src/gallium/drivers/radeonsi/radeonsi_shader.c
>> +++ b/src/gallium/drivers/radeonsi/radeonsi_shader.c
>> [...]
>> @@ -478,7 +472,6 @@ static void si_llvm_init_export_args(struct lp_build_tgsi_context *bld_base,
>>   		int cbuf = target - V_008DFC_SQ_EXP_MRT;
>>   
>>   		if (cbuf >= 0 && cbuf < 8) {
>> -			struct r600_context *rctx = si_shader_ctx->rctx;
>>   			compressed = (si_shader_ctx->key.export_16bpc >> cbuf) & 0x1;
>>   
>>   			if (compressed)
> Don't sneak in unrelated changes like this. You can push trivial fixes
> like this without review.

Ups, that was unintentional. I usually check for any warnings after 
coding something and didn't realized that this warning was there before 
I made any change. Going to fix it before committing.

Thx,
Christian.


More information about the mesa-dev mailing list