Re: [PATCH][LTO] Fix PR48437

2011-12-08 Thread Richard Guenther
On Wed, 7 Dec 2011, Diego Novillo wrote: > On Wed, Dec 7, 2011 at 11:16, Richard Guenther wrote: > > > I'm going to apply it tomorrow, when full testing hopefully finished > > Sure. But remember the zombie kitties! I have applied the fix for PR48437, the fix for PR49945 required adjustment as

Re: [PATCH][LTO] Fix PR48437

2011-12-07 Thread Diego Novillo
On Wed, Dec 7, 2011 at 11:16, Richard Guenther wrote: > I'm going to apply it tomorrow, when full testing hopefully finished Sure. But remember the zombie kitties! Diego.

Re: [PATCH][LTO] Fix PR48437

2011-12-07 Thread Richard Guenther
On Wed, 7 Dec 2011, Diego Novillo wrote: > On 12/07/11 10:54, Richard Guenther wrote: > > On Wed, 7 Dec 2011, Diego Novillo wrote: > > > > > On 12/07/11 10:46, Richard Guenther wrote: > > > > On Wed, 7 Dec 2011, Diego Novillo wrote: > > > > > > > > > On 12/07/11 09:52, Richard Guenther wrote: >

Re: [PATCH][LTO] Fix PR48437

2011-12-07 Thread Diego Novillo
On 12/07/11 10:54, Richard Guenther wrote: On Wed, 7 Dec 2011, Diego Novillo wrote: On 12/07/11 10:46, Richard Guenther wrote: On Wed, 7 Dec 2011, Diego Novillo wrote: On 12/07/11 09:52, Richard Guenther wrote: Index: gcc/lto-streamer-out.c =

Re: [PATCH][LTO] Fix PR48437

2011-12-07 Thread Richard Guenther
On Wed, 7 Dec 2011, Diego Novillo wrote: > On 12/07/11 10:46, Richard Guenther wrote: > > On Wed, 7 Dec 2011, Diego Novillo wrote: > > > > > On 12/07/11 09:52, Richard Guenther wrote: > > > > > > > Index: gcc/lto-streamer-out.c > > > >

Re: [PATCH][LTO] Fix PR48437

2011-12-07 Thread Diego Novillo
On 12/07/11 10:46, Richard Guenther wrote: On Wed, 7 Dec 2011, Diego Novillo wrote: On 12/07/11 09:52, Richard Guenther wrote: Index: gcc/lto-streamer-out.c === *** gcc/lto-streamer-out.c (revision 182081) --- gcc/lto-stream

Re: [PATCH][LTO] Fix PR48437

2011-12-07 Thread Richard Guenther
On Wed, 7 Dec 2011, Diego Novillo wrote: > On 12/07/11 09:52, Richard Guenther wrote: > > > Index: gcc/lto-streamer-out.c > > === > > *** gcc/lto-streamer-out.c (revision 182081) > > --- gcc/lto-streamer-out.c (working copy) > > **

Re: [PATCH][LTO] Fix PR48437

2011-12-07 Thread Diego Novillo
On 12/07/11 09:52, Richard Guenther wrote: Index: gcc/lto-streamer-out.c === *** gcc/lto-streamer-out.c (revision 182081) --- gcc/lto-streamer-out.c (working copy) *** tree_is_indexable (tree t) *** 129,134 ***

Re: [PATCH][LTO] Fix PR48437

2011-12-07 Thread Richard Guenther
On Wed, 7 Dec 2011, Jakub Jelinek wrote: > On Wed, Dec 07, 2011 at 03:52:35PM +0100, Richard Guenther wrote: > > *** gcc/testsuite/gcc.dg/guality/pr48437.c (revision 0) > > --- gcc/testsuite/gcc.dg/guality/pr48437.c (revision 0) > > *** > > *** 0 > > --- 1,15 > > + /* PR lt

Re: [PATCH][LTO] Fix PR48437

2011-12-07 Thread Jakub Jelinek
On Wed, Dec 07, 2011 at 03:52:35PM +0100, Richard Guenther wrote: > *** gcc/testsuite/gcc.dg/guality/pr48437.c(revision 0) > --- gcc/testsuite/gcc.dg/guality/pr48437.c(revision 0) > *** > *** 0 > --- 1,15 > + /* PR lto/48437 */ > + /* { dg-do run } */ > + /* { dg-optio

[PATCH][LTO] Fix PR48437

2011-12-07 Thread Richard Guenther
I'm testing the following patch for PR48437 - the C frontend generates decl copies for placing them in BLOCK_VARS for int test (void) { extern int f (void); return 0; } we currently end up putting those into the decl merging machinery, causing DECL_CHAIN re-use and subsequent crashes. By no