This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: C++ PATCH for c++/43075 (wrong-code on libstdc++ bind/ref2.cc testcase)
- From: Richard Guenther <richard dot guenther at gmail dot com>
- To: Jason Merrill <jason at redhat dot com>
- Cc: gcc-patches List <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 18 Feb 2010 00:00:15 +0100
- Subject: Re: C++ PATCH for c++/43075 (wrong-code on libstdc++ bind/ref2.cc testcase)
- References: <4B7C743B.6040802@redhat.com>
On Wed, Feb 17, 2010 at 11:56 PM, Jason Merrill <jason@redhat.com> wrote:
> Stores of empty classes are problematic for the back end, because they have
> non-zero TYPE_SIZE but consist entirely of tail padding, which might
> actually contain data from a derived class, so we don't want to touch it.
> ?The patch for 41058 fixed this for MODIFY_EXPRs, but in this PR we see that
> it can also occur for INIT_EXPRs. ?Rather than extend the 41058 patch to
> cover that case, I decided to avoid generating the offending copies in the
> first place.
Great!
Thanks,
Richard.
> Tested x86_64-pc-linux-gnu, applied to trunk.
>
>