Re: More of ipa-inline housekeeping

2011-04-17 Thread Jan Hubicka
> > > > I never really tuned the stack frame growth heuristics since it did not > > cause any problems > > in the benchmarks. On fortran this is quite different because of the large > > i/o blocks > > hitting it very commonly, so I will look into making it more permissive.   > > We definitely > >

Re: More of ipa-inline housekeeping

2011-04-17 Thread Richard Guenther
On Sun, Apr 17, 2011 at 10:35 AM, Jan Hubicka wrote: >> AFAICT revision 172430 fixed the original problem in pr45810: >> >> gfc -Ofast -fwhole-program fatigue.f90       : 6.301u 0.003s 0:06.30 >> gfc -Ofast -fwhole-program -flto fatigue.f90 : 6.263u 0.003s 0:06.26 >> >> However if I play with --pa

Re: More of ipa-inline housekeeping

2011-04-17 Thread Dominique Dhumieres
Honza, > I will also look into the estimate_size ICE reported today. This has been fixed by revision 172603 and now the thresholds are the same with/without -flto. For the fine tuning I have posted some results on pr48636. Cheers, Dominique

Re: More of ipa-inline housekeeping

2011-04-17 Thread Jan Hubicka
> AFAICT revision 172430 fixed the original problem in pr45810: > > gfc -Ofast -fwhole-program fatigue.f90 : 6.301u 0.003s 0:06.30 > gfc -Ofast -fwhole-program -flto fatigue.f90 : 6.263u 0.003s 0:06.26 > > However if I play with --param max-inline-insns-auto=*, I get > > gfc -Ofast -fwhole

Re: More of ipa-inline housekeeping

2011-04-16 Thread H.J. Lu
On Sat, Apr 16, 2011 at 3:42 PM, H.J. Lu wrote: > On Wed, Apr 13, 2011 at 3:20 PM, Jan Hubicka wrote: >> Hi, >> this patch moves inline_summary from field in cgraph_node into its own on >> side >> datastructure. This moves it from arcane decision of mine to split all IPA >> data >> into global/

Re: More of ipa-inline housekeeping

2011-04-16 Thread H.J. Lu
On Wed, Apr 13, 2011 at 3:20 PM, Jan Hubicka wrote: > Hi, > this patch moves inline_summary from field in cgraph_node into its own on side > datastructure. This moves it from arcane decision of mine to split all IPA > data > into global/local datas stored in common datastructure into the scheme w

Re: More of ipa-inline housekeeping

2011-04-15 Thread Dominique Dhumieres
I have forgotten to say that the 125 and 516 thresholds are for x86_64-apple-darwin10. On powerpc-apple-darwin9 they are repsectively 172 and 638. Dominique

Re: More of ipa-inline housekeeping

2011-04-15 Thread Dominique Dhumieres
AFAICT revision 172430 fixed the original problem in pr45810: gfc -Ofast -fwhole-program fatigue.f90 : 6.301u 0.003s 0:06.30 gfc -Ofast -fwhole-program -flto fatigue.f90 : 6.263u 0.003s 0:06.26 However if I play with --param max-inline-insns-auto=*, I get gfc -Ofast -fwhole-program --param

Re: More of ipa-inline housekeeping

2011-04-15 Thread Jan Hubicka
> 2011/4/15 Jan Hubicka : > >> > > >> > I fixed this on the and added sanity check that the fields are > >> > initialized. > >> > This has shown problem with early inliner iteration fixed thusly and > >> > fact that > >> > early inliner is attempting to compute overall growth at a time the > >>

Re: More of ipa-inline housekeeping

2011-04-15 Thread Richard Guenther
2011/4/15 Jan Hubicka : >> > >> > I fixed this on the and added sanity check that the fields are initialized. >> > This has shown problem with early inliner iteration fixed thusly and fact >> > that >> > early inliner is attempting to compute overall growth at a time the inline >> > parameters are

Re: More of ipa-inline housekeeping

2011-04-15 Thread Jan Hubicka
> > > > I fixed this on the and added sanity check that the fields are initialized. > > This has shown problem with early inliner iteration fixed thusly and fact > > that > > early inliner is attempting to compute overall growth at a time the inline > > parameters are not computed for functions no

Re: More of ipa-inline housekeeping

2011-04-14 Thread Richard Guenther
On Thu, Apr 14, 2011 at 12:20 AM, Jan Hubicka wrote: > Hi, > this patch moves inline_summary from field in cgraph_node into its own on side > datastructure. This moves it from arcane decision of mine to split all IPA > data > into global/local datas stored in common datastructure into the scheme

More of ipa-inline housekeeping

2011-04-13 Thread Jan Hubicka
Hi, this patch moves inline_summary from field in cgraph_node into its own on side datastructure. This moves it from arcane decision of mine to split all IPA data into global/local datas stored in common datastructure into the scheme we developed for new IPA passes some time ago. The advantage is