This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Add missing variable_size GTY annotations (was Re: on how to compile gcc-4.6 correctly?)
- From: "Dennis, CHENG Renquan" <crquan at fedoraproject dot org>
- To: Laurynas Biveinis <laurynas dot biveinis at gmail dot com>
- Cc: gcc-patches at gcc dot gnu dot org, Kai Ruottu <kai dot ruottu at wippies dot com>, gcc at gcc dot gnu dot org
- Date: Wed, 15 Sep 2010 00:59:18 +0800
- Subject: Re: [PATCH] Add missing variable_size GTY annotations (was Re: on how to compile gcc-4.6 correctly?)
- References: <AANLkTinvFhq6AyOw8p4a6+UL2F-SMf9bMS+N=p+Z607P@mail.gmail.com>
On Tue, Sep 14, 2010 at 9:17 PM, Laurynas Biveinis
<laurynas.biveinis@gmail.com> wrote:
> I have reproduced it and the patch below fixes the issue, sorry for
> breaking things. Dennis, could you see if it works for you?
>
> When gcc-core tarball is used without other frontends, gengtype does
> not get to see that lang_type is in fact variable_size and when the
> frontends are present, their variable_size annotations mask the fact
> that c-lang.h does not have it. This is something that really should
> be diagnosed by gengtype.
>
> I will apply it to trunk later as obvious.
Well, I have tested and verified that patch works well,
gcc-core-4.6-20100911 compiled well on my Ubuntu and Gentoo with my
default simplified configuration setting;
Tested-by: "Dennis, CHENG Renquan" <crquan@fedoraproject.org>
>
> gcc/ChangeLog:
>
> 2010-09-14 ÂLaurynas Biveinis Â<laurynas.biveinis@gmail.com>
>
> Â Â Â Â* c-lang.h (struct lang_type): Add variable_size GTY option.
>
> gcc/lto/ChangeLog:
>
> 2010-09-14 ÂLaurynas Biveinis Â<laurynas.biveinis@gmail.com>
>
> Â Â Â Â* lto-tree.h (struct lang_type): Add variable_size GTY option.