Re: [PATCH] Update optimization_default_node in ada (PR middle-end/64340)

2015-02-07 Thread Richard Biener
On February 7, 2015 11:55:23 AM CET, Eric Botcazou wrote: >> I think for using option processing langhooks it would require >massive Ada >> FE surgery, the thing is that right now all the option processing is >> performed, then at some later point the TU is parsed, then depending >on what >> is s

Re: [PATCH] Update optimization_default_node in ada (PR middle-end/64340)

2015-02-07 Thread Eric Botcazou
> I think for using option processing langhooks it would require massive Ada > FE surgery, the thing is that right now all the option processing is > performed, then at some later point the TU is parsed, then depending on what > is seen in there the options are tweaked and finally everything is han

Re: [PATCH] Update optimization_default_node in ada (PR middle-end/64340)

2015-02-07 Thread Jakub Jelinek
On Sat, Feb 07, 2015 at 11:28:54AM +0100, Richard Biener wrote: > On February 6, 2015 9:26:29 PM CET, Jakub Jelinek wrote: > >Hi! > > > >As mentioned in the PR, the problem here is that the Ada FE needs to > >modify > >global_options after toplevel.c (process_options), but as for LTO we > >now use

Re: [PATCH] Update optimization_default_node in ada (PR middle-end/64340)

2015-02-07 Thread Richard Biener
On February 6, 2015 9:26:29 PM CET, Jakub Jelinek wrote: >Hi! > >As mentioned in the PR, the problem here is that the Ada FE needs to >modify >global_options after toplevel.c (process_options), but as for LTO we >now use >optimization_{default,current}_node for options of functions without >specif

[PATCH] Update optimization_default_node in ada (PR middle-end/64340)

2015-02-06 Thread Jakub Jelinek
Hi! As mentioned in the PR, the problem here is that the Ada FE needs to modify global_options after toplevel.c (process_options), but as for LTO we now use optimization_{default,current}_node for options of functions without specifial optimization node, it means the changed options aren't reflect