Re: [patch] Call free_after_parsing earlier

2012-07-16 Thread Richard Guenther
On Fri, Jul 13, 2012 at 4:33 PM, Steven Bosscher wrote: > On Thu, Jul 12, 2012 at 9:21 AM, Richard Guenther > wrote: >> On Wed, Jul 11, 2012 at 9:39 PM, Steven Bosscher >> wrote: >>> Hello, >>> >>> GCC calls free_after_parsing in rest_of_clean_state. >>> That's way too late, it can be done in f

Re: [patch] Call free_after_parsing earlier

2012-07-13 Thread Steven Bosscher
On Thu, Jul 12, 2012 at 9:21 AM, Richard Guenther wrote: > On Wed, Jul 11, 2012 at 9:39 PM, Steven Bosscher > wrote: >> Hello, >> >> GCC calls free_after_parsing in rest_of_clean_state. >> That's way too late, it can be done in free_lang_data_in_cgraph instead. > > But that's only called with -f

Re: [patch] Call free_after_parsing earlier

2012-07-12 Thread Richard Guenther
On Wed, Jul 11, 2012 at 9:39 PM, Steven Bosscher wrote: > Hello, > > GCC calls free_after_parsing in rest_of_clean_state. > That's way too late, it can be done in free_lang_data_in_cgraph instead. But that's only called with -flto ... I think it should be called in cgraph_finalize_function instea

[patch] Call free_after_parsing earlier

2012-07-11 Thread Steven Bosscher
Hello, GCC calls free_after_parsing in rest_of_clean_state. That's way too late, it can be done in free_lang_data_in_cgraph instead. While there, I noticed a silly loop in final.c, and cleaned that up too. Bootstrapped&tested on x86_64-unknown-linux-gnu. OK for trunk? Ciao! Steven * fi