PATCH: Inliner: Don't generate memcpy calls for byval readonly parameters

Chandler Carruth chandlerc at google.com
Wed Oct 23 17:10:23 PDT 2013


On Wed, Oct 23, 2013 at 3:54 PM, Tom Stellard <tom at stellard.net> wrote:

> On Wed, Oct 23, 2013 at 05:52:26AM -0700, Vincent Lejeune wrote:
> > Thank for adding some test to my patch !
> > However is it possible to change the name of the HandleByValArgument ?
> > I used ArgAsArgument but actually Arg and ArgAsArgument represent 2
> different things,
> > Arg is the value passed to the function call, while ArgAsArgument is
> indeed the argument
> > in the called signature. I think renaming Arg to "PassedValue" and
> ArgAsValue to
> > "ArgumentSignature" would make the description of both parameters more
> self
> > explanatory.
> >
>
> Here is an updated version of the patch, does this look OK to everyone?


LGTM
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131023/3ceab9f2/attachment.html>


More information about the llvm-commits mailing list