[PATCH] Have 'this'-returning constructors and destructors to take advantage of the new backend 'returned' attribute

John McCall rjmccall at apple.com
Tue Jun 18 08:43:53 PDT 2013


On Jun 17, 2013, at 1:40 PM, Stephen Lin <swlin at apple.com> wrote:

> Ping :)
> <this-return.patch>

Sorry about this delay.  Looks great, thanks.

John.

> On Jun 5, 2013, at 8:52 AM, Stephen Lin <swlin at apple.com> wrote:
> 
>>> Well, they're serving the same purpose.
>>> 
>>> Could you combine them with the changes I've requested and send that out?  And then I'll just review that.
>>> 
>>> John.
>> 
>> Oh, just to be clear since it got truncated a few replies ago, the split is such that the first is basically reversion of the existing implementation using CalleeWithThisReturn and the second is the new implementation. Thought it might be helpful to keep them separate, but no worries if don't agree :)
>> 
>> Here's a consolidated patch with the suggested changes. Let me know if there's anything else I should fix/improve :)
>> 
>> -Stephen
>> 
>> <this-return.patch>
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130618/ba719f8a/attachment.html>


More information about the cfe-commits mailing list