Re: PING^2: [PATCH] Limit alignment on error_mark_node variable

2015-10-20 Thread Jan Hubicka
> >> > >> This patch avoids calling varpool_node::finalize_decl on error_mark_node > >> type decls. Does it make sense? Yep, i guess this is fine. I wonder why error_mark_node needs to be in local decl list? :) honza > >> > >> > >> H.J. > >> -- > >> gcc/ > >> > >> PR target/66810 > >>

Re: PING^2: [PATCH] Limit alignment on error_mark_node variable

2015-10-19 Thread Bernd Schmidt
On 10/20/2015 12:07 AM, H.J. Lu wrote: On Thu, Jul 09, 2015 at 03:57:31PM +0200, Richard Biener wrote: I don't see why the C FE would need to invoke finish_decl twice here. [...] So I'd rather have the C frontend not invoke rest_of_decl_compilation on this in the first place. Your patch still

PING^2: [PATCH] Limit alignment on error_mark_node variable

2015-10-19 Thread H.J. Lu
PING On Wed, Sep 30, 2015 at 9:13 AM, H.J. Lu wrote: > PING > > On Fri, Jul 10, 2015 at 5:19 AM, H.J. Lu wrote: >> On Thu, Jul 09, 2015 at 03:57:31PM +0200, Richard Biener wrote: >>> On Thu, Jul 9, 2015 at 1:08 PM, H.J. Lu wrote: >>> > On Thu, Jul 9, 2015 at 2:54 AM, Richard Biener >>> > wrote