This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: C++ PATCH for c++/70259 (-flifetime-dse vs. empty bases)
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Jason Merrill <jason at redhat dot com>
- Cc: gcc-patches List <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 16 Mar 2016 23:50:21 +0100
- Subject: Re: C++ PATCH for c++/70259 (-flifetime-dse vs. empty bases)
- Authentication-results: sourceware.org; auth=none
- References: <56E9AA2D dot 8000901 at redhat dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Wed, Mar 16, 2016 at 02:47:09PM -0400, Jason Merrill wrote:
> The constructor for an empty class can't do the -flifetime-dse clobber
> because when the class is used as a base it might be assigned the same
> offset as a real base, so the clobber would mess with real data.
Isn't this needed also for the begin_destructor_body case?
I mean can't it clobber prematurely something that shouldn't be clobbered
yet?
> commit e1a5f038350d1881153d8f65359bd883f7452237
> Author: Jason Merrill <jason@redhat.com>
> Date: Wed Mar 16 13:46:32 2016 -0400
>
> PR c++/70259
> * decl.c (start_preparsed_function): Don't clobber an empty base.
Jakub