
<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Apr 11, 2014, at 3:40 PM, Jan Vesely <<a href="mailto:jan.vesely@rutgers.edu">jan.vesely@rutgers.edu</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">On Thu, 2014-04-10 at 10:13 -0400, Tom Stellard wrote:<br><blockquote type="cite">On Tue, Apr 08, 2014 at 08:23:24PM -0400, Jan Vesely wrote:<br><blockquote type="cite">Hi,<br><br>I finally got some time to improve the tests. My original issue with<br>*_generated piglits got fixed in the mean time. (My bets are it was<br>R600: Correct opcode for BFE_INT :).<br><br>I tried to use your latest patch to base the tests on but it did not<br>apply cleanly on master.<br><br>I have attached another patch(0001), that is based on your idea and<br>managed to get llvm do the scalarization, so no custom procedures are<br>necessary for EG hw.<br><br></blockquote><br>These patches look good, I will commit them.<br></blockquote><br>Thanks, I have attached a cleaned up v2 of the first patch. It moves the<br>code out of AMDGPU shared class, and makes it more obvious there is no<br>change for SI.<br><br>regards,<br>Jan<br><br></div></blockquote><br></div><div>I’ve committed these as r206348 and r206349</div><div><br></div><div>-Matt</div></body></html>