This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: RFA: PATCH to store_field for storing a CONSTRUCTOR into a base subobject

From: Jeff Law <law at redhat dot com>
To: Jason Merrill <jason at redhat dot com>, gcc-patches List <gcc-patches at gcc dot gnu dot org>
Date: Fri, 9 Oct 2015 15:02:33 -0600
Subject: Re: RFA: PATCH to store_field for storing a CONSTRUCTOR into a base subobject
Authentication-results: sourceware.org; auth=none
References: <5615D805 dot 30806 at redhat dot com>

On 10/07/2015 08:42 PM, Jason Merrill wrote:

While looking at another issue I noticed that in g++.dg/init/vbase1.C
the Diamond(int) constructor was unnecessarily storing a CONSTRUCTOR
into a stack temporary and then copying it into the SubB base subobject
rather than directly storing the CONSTRUCTOR.  It was doing this because
the base subobject is smaller than a complete object of SubB, but we
already know how to store a CONSTRUCTOR into a space smaller than it
would normally take without needing to introduce an extra temporary.
This patch fixes that.

Tested x86_64-pc-linux-gnu.  OK for trunk?

OK.
jeff

References:
- RFA: PATCH to store_field for storing a CONSTRUCTOR into a base subobject
  - From: Jason Merrill

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]