[PATCH] Allow CodeGen to take advantage of functions that always return arguments

Stephen Lin swlin at post.harvard.edu
Fri Apr 19 01:29:24 PDT 2013


On Thu, Apr 18, 2013 at 11:23 PM, Evan Cheng <evan.cheng at apple.com> wrote:
> Thanks. One suggestion. Next time please separate out stylistic changes into a separate patch.
>
> Evan
>
> Sent from my iPad

OK, I removed some extraneous changes and updated the patch to be
based against a mostly non-recursive implementation of sameNoopInput:

    http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20130415/172039.html

I've also added the restriction that 'returned' can only be on one
parameter to allow sameNoopInput to see through 'returned' without
recursion or extra bookkeeping. There's a minor tack-on bug fix;
apparently the verifier check that 'nest' is only on one parameter was
never working correctly...I'm willing to split that off to a separate
patch if that's preferable though.

The other patch listed is my original e-mail has been committed so is
no longer an uncommitted dependency.

Stephen
-------------- next part --------------
A non-text attachment was scrubbed...
Name: return-argument.patch
Type: application/octet-stream
Size: 23901 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130419/e37960cc/attachment.obj>


More information about the llvm-commits mailing list