Re: Minor fixes to ipa-inline-analysis.c

2012-11-26 Thread Jan Hubicka
> > @@ -692,12 +692,14 @@ account_size_time (struct inline_summary > > found = true; > > break; > >} > > - if (i == 32) > > + if (i == 256) > > Shouldn't this be a --param? You are probably right. ipa-inline-analysis has few hard wired constants in it but mostly to al

Re: Minor fixes to ipa-inline-analysis.c

2012-11-25 Thread Richard Biener
On Wed, Nov 7, 2012 at 10:25 AM, Jan Hubicka wrote: > Hi, > while analyzing c-ray I noticed two issues. First is that I originally set > number > of size/time entries to 32. Once we reach this limit we conservatively > account > everything as unconditional. This limit is not met on relatively

Minor fixes to ipa-inline-analysis.c

2012-11-07 Thread Jan Hubicka
Hi, while analyzing c-ray I noticed two issues. First is that I originally set number of size/time entries to 32. Once we reach this limit we conservatively account everything as unconditional. This limit is not met on relatively simple testcases, like ray-sphere. The reason is that aggregate