Re: Inliner heuristics facelift

2011-04-19 Thread H.J. Lu
On Tue, Apr 19, 2011 at 11:30 AM, Xinliang David Li wrote: > Is change tested this with SPEC?  5 or 6 SPEC2k programs ICE with the > following error when built with FDO. > > internal compiler error: in inline_small_functions, at ipa-inline.c:1361 > Please submit a full bug report, > with prep

Re: Inliner heuristics facelift

2011-04-19 Thread H.J. Lu
On Sun, Apr 17, 2011 at 7:14 AM, Jan Hubicka wrote: > Hi, > there is now tracking PR48636 for fortran inlining issues.  I would suggest > reporting problems like this there or PR that blocks it instead of sending it > to different threads so we have track of them.  It is quite important to get > s

Re: Inliner heuristics facelift

2011-04-17 Thread Jan Hubicka
> > For me, I get everything inlined except for: > > Oops! looking at my mail, I realized that I did not timed fatigue, but ac. > With the patch the only difference for fatigue is that the threshold for > full inlining decreased from 125 to 113. See, we make progress. 10 more patches like this an

Re: Inliner heuristics facelift

2011-04-17 Thread Dominique Dhumieres
> For me, I get everything inlined except for: Oops! looking at my mail, I realized that I did not timed fatigue, but ac. With the patch the only difference for fatigue is that the threshold for full inlining decreased from 125 to 113. Sorry for the noise. Dominique

Re: Inliner heuristics facelift

2011-04-17 Thread Jan Hubicka
> Honza, > > Your patch seems to make --param max-inline-insns-auto= ineffective: > > gfc -Ofast -funroll-loops -ftree-loop-linear -fomit-frame-pointer --param > max-inline-insns-auto=4000 -fwhole-program -fstack-arrays ac.f90 > 8.105u 0.005s 0:08.11 99.8% 0+0k 0+5io 0pf+0w > > while the timi

Re: Inliner heuristics facelift

2011-04-17 Thread Dominique Dhumieres
Honza, Your patch seems to make --param max-inline-insns-auto= ineffective: gfc -Ofast -funroll-loops -ftree-loop-linear -fomit-frame-pointer --param max-inline-insns-auto=4000 -fwhole-program -fstack-arrays ac.f90 8.105u 0.005s 0:08.11 99.8% 0+0k 0+5io 0pf+0w while the timing was ~2.7s wit