[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