PATCH: add NVPTX test for a function call with a populated local buffer

Eli Bendersky eliben at google.com
Mon Mar 24 09:59:34 PDT 2014


On Mon, Mar 24, 2014 at 9:47 AM, Justin Holewinski
<jholewinski at nvidia.com>wrote:

>  Yes, looks good, thanks!
>

Thanks. r204624.


>
>  On 03/24/2014 12:44 PM, Eli Bendersky wrote:
>
>
>
>
> On Thu, Mar 20, 2014 at 10:33 AM, Eli Bendersky <eliben at google.com> wrote:
>
>>
>>
>>
>>  On Thu, Mar 20, 2014 at 10:22 AM, Justin Holewinski <
>> jholewinski at nvidia.com> wrote:
>>
>>>  Where are %rl1 and %rl2 coming from in the st.param instructions?  I
>>> would expect the %rl2 to be
>>>
>>> %rl[[SP_REG]]
>>>
>>>
>>  Oops, good catch. I forgot to propagate the regex-ed names. Updated
>> patch attached.
>>
>>
>  Justin, revised patch LGTY?
>
>  Eli
>
>
>
>>   Eli
>>
>>
>>
>>
>>>
>>>
>>> On 03/20/2014 01:15 PM, Eli Bendersky wrote:
>>>
>>> Hi Justin,
>>>
>>>  PTAL at the attached test I'd like to add to test/CodeGen/NVPTX to
>>> make sure calls with "alloca buffer" arguments get lowered as expected.
>>>
>>>  Eli
>>>
>>>
>>>   ------------------------------
>>>  This email message is for the sole use of the intended recipient(s)
>>> and may contain confidential information.  Any unauthorized review, use,
>>> disclosure or distribution is prohibited.  If you are not the intended
>>> recipient, please contact the sender by reply email and destroy all copies
>>> of the original message.
>>>  ------------------------------
>>>
>>
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140324/08676d6a/attachment.html>


More information about the llvm-commits mailing list