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

John McCall rjmccall at apple.com
Tue Jun 4 18:15:15 PDT 2013


On Jun 3, 2013, at 8:11 PM, Stephen Lin <swlin at apple.com> wrote:
> Here's a freshly rebased pair of patches.
> <this-return1.patch><this-return2.patch>

I don't understand why EmitConstructorCall needs to take a ReturnValueSlot or return an RValue.  There's one call site, and it passes an empty ReturnValueSlot and ignores the result. This seems to just be cluttering the API.

Is there a reason why these patches are separated?  (Did I ask for that for some reason?)

John.



More information about the cfe-commits mailing list