[PATCH] R600: Make sign_extend_inreg legal.

Matt Arsenault arsenm2 at gmail.com
Tue Apr 15 18:48:41 PDT 2014


On Apr 11, 2014, at 3:40 PM, Jan Vesely <jan.vesely at rutgers.edu> wrote:

> On Thu, 2014-04-10 at 10:13 -0400, Tom Stellard wrote:
>> On Tue, Apr 08, 2014 at 08:23:24PM -0400, Jan Vesely wrote:
>>> Hi,
>>> 
>>> I finally got some time to improve the tests. My original issue with
>>> *_generated piglits got fixed in the mean time. (My bets are it was
>>> R600: Correct opcode for BFE_INT :).
>>> 
>>> I tried to use your latest patch to base the tests on but it did not
>>> apply cleanly on master.
>>> 
>>> I have attached another patch(0001), that is based on your idea and
>>> managed to get llvm do the scalarization, so no custom procedures are
>>> necessary for EG hw.
>>> 
>> 
>> These patches look good, I will commit them.
> 
> Thanks, I have attached a cleaned up v2 of the first patch. It moves the
> code out of AMDGPU shared class, and makes it more obvious there is no
> change for SI.
> 
> regards,
> Jan
> 

I’ve committed these as r206348 and r206349

-Matt
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140415/16454a36/attachment.html>


More information about the llvm-commits mailing list