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