[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