[cfe-commits] [PATCH] Fix struct assignment from "piecemeal" aggregates

John McCall rjmccall at apple.com
Fri Jun 17 14:32:08 PDT 2011

On Jun 17, 2011, at 2:13 PM, Eli Friedman wrote:
> On Fri, Jun 17, 2011 at 11:21 AM, Douglas Gregor <dgregor at apple.com> wrote:
>> The attached patch fixes the root of the problem: by detecting when the rhs is something that will build an aggregate piecemeal, we can generate safer code that creates a temporary for the rhs and then copies it over. In the common case (rhs builds a full aggregate value in one shot), we still perform the optimization.
>> Comments appreciated!
> I'm sort of uncomfortable with the blacklist approach... are you sure
> you're actually catching all the unsafe cases?

I agree;  we should be white-listing expressions that are actually safe
to emit directly like this.  The only cases that come to mind would be
(expressions that produce their value via one of)
  an lvalue-to-rvalue conversion 
  a rvalue member expression

Alternatively, we could propagate a "result is potentially aliased"
bit in AggValueSlot.


More information about the cfe-commits mailing list