Re: [Mesa-dev] [PATCH 0/2] r600g: Pair of small code clean ups with TGSI

[Eric Anholt]

Rhys Kidd  writes:

> Having run Mesa through Clang on Eric Anholt's Travis harness, these small
> code clean ups improve readability of TGSI code in r600g and may avoid
> future problems.
>
> Series also can be found at:
> https://github.com/Echelon9/mesa/tree/fix/r600g-cleanup-tgsi-opcodes
>
> I don't have the hardware to test this so would appreciated Tested-by's.
>
> I do not have commit rights to fd.o so after R-B would appreciate if
> the reviewer(s) could push to master.

Applied the tags and pushed.  Thanks!


signature.asc
Description: PGP signature
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 0/2] r600g: Pair of small code clean ups with TGSI

[James Harvey]

Hi Rhys,

 I ran piglit on my Evergreen HD5850 with your patches.

No regressions here.

Tested-by: James Harvey 

Thanks,
James


On 08/27/2016 09:05 AM, Rhys Kidd wrote:

Having run Mesa through Clang on Eric Anholt's Travis harness, these small
code clean ups improve readability of TGSI code in r600g and may avoid
future problems.

Series also can be found at:
https://github.com/Echelon9/mesa/tree/fix/r600g-cleanup-tgsi-opcodes

I don't have the hardware to test this so would appreciated Tested-by's.

I do not have commit rights to fd.o so after R-B would appreciate if
the reviewer(s) could push to master.

Rhys Kidd (2):
  r600g: Avoid duplicated initialization of TGSI_OPCODE_DFMA
  r600g: Clean up defined magic numbers for TGSI opcodes

 src/gallium/drivers/r600/r600_shader.c | 18 +-
 src/gallium/include/pipe/p_shader_tokens.h |  1 +
 2 files changed, 10 insertions(+), 9 deletions(-)


___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 0/2] r600g: Pair of small code clean ups with TGSI

[Rhys Kidd]

On Monday, August 29, 2016, James Harvey  wrote:

> Hi Rhys,
>
>  I ran piglit on my Evergreen HD5850 with your patches.
>
> No regressions here.
>
> Tested-by: James Harvey 


Thanks James.


>
> Thanks,
> James
>
>
> On 08/27/2016 09:05 AM, Rhys Kidd wrote:
>
>> Having run Mesa through Clang on Eric Anholt's Travis harness, these small
>> code clean ups improve readability of TGSI code in r600g and may avoid
>> future problems.
>>
>> Series also can be found at:
>> https://github.com/Echelon9/mesa/tree/fix/r600g-cleanup-tgsi-opcodes
>>
>> I don't have the hardware to test this so would appreciated Tested-by's.
>>
>> I do not have commit rights to fd.o so after R-B would appreciate if
>> the reviewer(s) could push to master.
>>
>> Rhys Kidd (2):
>>   r600g: Avoid duplicated initialization of TGSI_OPCODE_DFMA
>>   r600g: Clean up defined magic numbers for TGSI opcodes
>>
>>  src/gallium/drivers/r600/r600_shader.c | 18 +-
>>  src/gallium/include/pipe/p_shader_tokens.h |  1 +
>>  2 files changed, 10 insertions(+), 9 deletions(-)
>>
>>
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH 0/2] r600g: Pair of small code clean ups with TGSI

[Rhys Kidd]

Having run Mesa through Clang on Eric Anholt's Travis harness, these small
code clean ups improve readability of TGSI code in r600g and may avoid
future problems.

Series also can be found at:
https://github.com/Echelon9/mesa/tree/fix/r600g-cleanup-tgsi-opcodes

I don't have the hardware to test this so would appreciated Tested-by's.

I do not have commit rights to fd.o so after R-B would appreciate if
the reviewer(s) could push to master.

Rhys Kidd (2):
  r600g: Avoid duplicated initialization of TGSI_OPCODE_DFMA
  r600g: Clean up defined magic numbers for TGSI opcodes

 src/gallium/drivers/r600/r600_shader.c | 18 +-
 src/gallium/include/pipe/p_shader_tokens.h |  1 +
 2 files changed, 10 insertions(+), 9 deletions(-)

-- 
2.7.4

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev