[Mesa-dev] [PATCH] ir_to_mesa: fix copy propagation

Eric Anholt eric at anholt.net
Wed Jan 26 22:34:06 PST 2011


On Wed, 26 Jan 2011 13:30:04 +0100, Christoph Bumiller <e0425955 at student.tuwien.ac.at> wrote:
> The current copy propagation code would propagate TEMP[0].x from (6)
> into TEMP[1].x from (8) in the following, which is clearly wrong:
> 
>   6:   MOV TEMP[1].x, TEMP[0].xxxx
>   7:   MOV TEMP[0].x, TEMP[0].yyyy
>   8:   MOV TEMP[0].y, TEMP[1].xxxx
>   9:   ADD TEMP[1].x, TEMP[0].zzzz, IMM[0].yyyy
>  10:   MOV TEMP[0].z, TEMP[1].xxxx
> 
> Possible patch and a small program to check this attached.
> Tested with llvmpipe.

Thanks for catching this!  Could you cut it down to a nice small
shader_test for piglit so we can regression test?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20110127/fc1a2b82/attachment.pgp>


More information about the mesa-dev mailing list