Re: [PATCH] Fix LTO option streaming

2015-04-27 Thread Richard Biener
On Thu, 23 Apr 2015, Richard Biener wrote: > On Thu, 23 Apr 2015, Jan Hubicka wrote: > > > > > > > It looks like when transitioning to using target and optimization > > > option nodes for compile-time to link-time option streaming you > > > didn't adjust lto-opts.c nor lto-wrapper.c. The follow

Re: [PATCH] Fix LTO option streaming

2015-04-23 Thread Jakub Jelinek
On Thu, Apr 23, 2015 at 05:16:19PM +0200, Jan Hubicka wrote: > > the target option side (for SWITCHABLE_TARGET). Do not record > > any target options in the lto_opts section. > > > > Honza - I suppose we don't have any testcase that this works, I'll > > try to come up with sth. This also looks l

Re: [PATCH] Fix LTO option streaming

2015-04-23 Thread Richard Biener
On Thu, 23 Apr 2015, Jan Hubicka wrote: > > > > It looks like when transitioning to using target and optimization > > option nodes for compile-time to link-time option streaming you > > didn't adjust lto-opts.c nor lto-wrapper.c. The following fixes > > Yep, I assumed that lto-wrapper's merging

Re: [PATCH] Fix LTO option streaming

2015-04-23 Thread Jan Hubicka
> > It looks like when transitioning to using target and optimization > option nodes for compile-time to link-time option streaming you > didn't adjust lto-opts.c nor lto-wrapper.c. The following fixes Yep, I assumed that lto-wrapper's merging is now redundant for optimization options, while it