<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"> <html> <head> <meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type"> </head> <body bgcolor="#ffffff" text="#000000"> On 27.01.2011 23:20, Christoph Bumiller wrote: <blockquote cite="mid:4D41EFC3.6080007@student.tuwien.ac.at" type="cite"> <pre wrap="">On 27.01.2011 07:34, Eric Anholt wrote: </pre> <blockquote type="cite"> <pre wrap="">On Wed, 26 Jan 2011 13:30:04 +0100, Christoph Bumiller <a class="moz-txt-link-rfc2396E" href="mailto:e0425955@student.tuwien.ac.at"><e0425955@student.tuwien.ac.at></a> wrote: </pre> <blockquote type="cite"> <pre wrap="">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. </pre> </blockquote> <pre wrap="">Thanks for catching this! Could you cut it down to a nice small shader_test for piglit so we can regression test? </pre> </blockquote> <pre wrap="">Here you are, a piglit test. </pre> </blockquote> Sorry, didn't (consciously) know about the shader_runner, here's a shader_test instead.<br> <blockquote cite="mid:4D41EFC3.6080007@student.tuwien.ac.at" type="cite"> <pre wrap=""></pre> <pre wrap=""> <fieldset class="mimeAttachmentHeader"></fieldset> _______________________________________________ mesa-dev mailing list <a class="moz-txt-link-abbreviated" href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a> <a class="moz-txt-link-freetext" href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a> </pre> </blockquote> <br> </body> </html>