[lto] Merge streamer hooks from pph branch. (issue4568043)

2011-05-31 Thread Diego Novillo
This patch merges the LTO streamer hooks from the pph branch. These hooks are meant to separate streaming work that is specific to GIMPLE from other modules that may want to stream their own data structures. In the PPH branch, we are using the streamer to save front end data structures, so there

Re: [lto] Merge streamer hooks from pph branch. (issue4568043)

2011-06-01 Thread Richard Guenther
On Tue, 31 May 2011, Diego Novillo wrote: > > This patch merges the LTO streamer hooks from the pph branch. > > These hooks are meant to separate streaming work that is specific to > GIMPLE from other modules that may want to stream their own data > structures. In the PPH branch, we are using t

Re: [lto] Merge streamer hooks from pph branch. (issue4568043)

2011-06-01 Thread Diego Novillo
On Wed, Jun 1, 2011 at 08:07, Richard Guenther wrote: >>  static void cgraph_expand_all_functions (void); >>  static void cgraph_mark_functions_to_output (void); >> @@ -1092,6 +1093,10 @@ cgraph_finalize_compilation_unit (void) >>  { >>    timevar_push (TV_CGRAPH); >> >> +  /* If LTO is enabled,

Re: [lto] Merge streamer hooks from pph branch. (issue4568043)

2011-06-01 Thread Richard Guenther
On Wed, 1 Jun 2011, Diego Novillo wrote: > On Wed, Jun 1, 2011 at 08:07, Richard Guenther wrote: > > >>  static void cgraph_expand_all_functions (void); > >>  static void cgraph_mark_functions_to_output (void); > >> @@ -1092,6 +1093,10 @@ cgraph_finalize_compilation_unit (void) > >>  { > >>    t

Re: [lto] Merge streamer hooks from pph branch. (issue4568043)

2011-06-04 Thread Diego Novillo
On Wed, Jun 1, 2011 at 15:19, Richard Guenther wrote: > Yes, I see no benefit of using a global function to get access > to the address of a global variable. There is the minor benefit of being able to control access to it, but I don't have a really convincing reason to give you, so I changed i

Re: [lto] Merge streamer hooks from pph branch. (issue4568043)

2011-06-06 Thread Richard Guenther
On Sat, 4 Jun 2011, Diego Novillo wrote: > On Wed, Jun 1, 2011 at 15:19, Richard Guenther wrote: > > > Yes, I see no benefit of using a global function to get access > > to the address of a global variable. > > There is the minor benefit of being able to control access to it, but > I don't hav

Re: [lto] Merge streamer hooks from pph branch. (issue4568043)

2011-06-06 Thread Diego Novillo
On Mon, Jun 6, 2011 at 10:50, Richard Guenther wrote: > Do you remember if it was only void_zero_node that causes problems? > We could just special-case it in > lto_input_integer_cst/lto_output_integer_cst.  Or even fix the C family > frontends to not create or use this strange node.  It seems to