On Monday, August 29, 2016, James Harvey <<a href="mailto:lothmordor@gmail.com">lothmordor@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Rhys,<br>
<br>
 I ran piglit on my Evergreen HD5850 with your patches.<br>
<br>
No regressions here.<br>
<br>
Tested-by: James Harvey <<a>lothmordor@gmail.com</a>></blockquote><div><br></div><div>Thanks James.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks,<br>
James<br>
<br>
<br>
On 08/27/2016 09:05 AM, Rhys Kidd wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Having run Mesa through Clang on Eric Anholt's Travis harness, these small<br>
code clean ups improve readability of TGSI code in r600g and may avoid<br>
future problems.<br>
<br>
Series also can be found at:<br>
<a href="https://github.com/Echelon9/mesa/tree/fix/r600g-cleanup-tgsi-opcodes" target="_blank">https://github.com/Echelon9/me<wbr>sa/tree/fix/r600g-cleanup-tgsi<wbr>-opcodes</a><br>
<br>
I don't have the hardware to test this so would appreciated Tested-by's.<br>
<br>
I do not have commit rights to fd.o so after R-B would appreciate if<br>
the reviewer(s) could push to master.<br>
<br>
Rhys Kidd (2):<br>
  r600g: Avoid duplicated initialization of TGSI_OPCODE_DFMA<br>
  r600g: Clean up defined magic numbers for TGSI opcodes<br>
<br>
 src/gallium/drivers/r600/<wbr>r600_shader.c     | 18 +++++++++---------<br>
 src/gallium/include/pipe/p_sh<wbr>ader_tokens.h |  1 +<br>
 2 files changed, 10 insertions(+), 9 deletions(-)<br>
<br>
</blockquote>
</blockquote>