PATCH: add NVPTX test for a function call with a populated local buffer
Justin Holewinski
jholewinski at nvidia.com
Mon Mar 24 09:47:41 PDT 2014
Yes, looks good, thanks!
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
> <mailto:eliben at google.com>> wrote:
>
>
>
>
> On Thu, Mar 20, 2014 at 10:22 AM, Justin Holewinski
> <jholewinski at nvidia.com <mailto: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/746090ba/attachment.html>
More information about the llvm-commits
mailing list