Re: [PATCH][RFC] Re-write LTO option merging

2011-10-29 Thread Richard Guenther
On Fri, Oct 28, 2011 at 4:42 PM, Diego Novillo wrote: > On 11-10-27 01:46 , Richard Guenther wrote: >> >> On Wed, 26 Oct 2011, Richard Guenther wrote: >> >>> >>> This completely rewrites LTO option merging.  At compile (uselessly >>> now at WPA?) time we now stream a COLLECT_GCC_OPTIONS like strin

Re: [PATCH][RFC] Re-write LTO option merging

2011-10-28 Thread Diego Novillo
On 11-10-27 01:46 , Richard Guenther wrote: On Wed, 26 Oct 2011, Richard Guenther wrote: This completely rewrites LTO option merging. At compile (uselessly now at WPA?) time we now stream a COLLECT_GCC_OPTIONS like string as it comes from argv of the compiler binary. Those options are read i

Re: [PATCH][RFC] Re-write LTO option merging

2011-10-27 Thread Richard Guenther
On Wed, 26 Oct 2011, Joseph S. Myers wrote: > On Wed, 26 Oct 2011, Richard Guenther wrote: > > > Index: trunk/gcc/opts-global.c > > === > > *** trunk.orig/gcc/opts-global.c2011-10-26 13:46:24.0 +0200 > > --- trunk/gcc/opt

Re: [PATCH][RFC] Re-write LTO option merging

2011-10-27 Thread Richard Guenther
On Wed, 26 Oct 2011, Richard Guenther wrote: > > This completely rewrites LTO option merging. At compile (uselessly > now at WPA?) time we now stream a COLLECT_GCC_OPTIONS like string > as it comes from argv of the compiler binary. Those options are > read in by the LTO driver (lto-wrapper), me

Re: [PATCH][RFC] Re-write LTO option merging

2011-10-27 Thread Richard Guenther
On Wed, 26 Oct 2011, Mike Stump wrote: > On Oct 26, 2011, at 6:10 AM, Richard Guenther wrote: > > This completely rewrites LTO option merging. > > > Any comments? > > Wondering if It breaks darwin? > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50876 > > We have: > >%{Zmultiply_defined*:-

Re: [PATCH][RFC] Re-write LTO option merging

2011-10-26 Thread Mike Stump
On Oct 26, 2011, at 6:10 AM, Richard Guenther wrote: > This completely rewrites LTO option merging. > Any comments? Wondering if It breaks darwin? http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50876 We have: %{Zmultiply_defined*:-multiply_defined %*} in LINK_SPEC. and: ; Various linker o

Re: [PATCH][RFC] Re-write LTO option merging

2011-10-26 Thread Joseph S. Myers
On Wed, 26 Oct 2011, Richard Guenther wrote: > Index: trunk/gcc/opts-global.c > === > *** trunk.orig/gcc/opts-global.c 2011-10-26 13:46:24.0 +0200 > --- trunk/gcc/opts-global.c 2011-10-26 13:48:42.0 +0200 >

[PATCH][RFC] Re-write LTO option merging

2011-10-26 Thread Richard Guenther
This completely rewrites LTO option merging. At compile (uselessly now at WPA?) time we now stream a COLLECT_GCC_OPTIONS like string as it comes from argv of the compiler binary. Those options are read in by the LTO driver (lto-wrapper), merged into a single set (very simple merge function right