Re: [PATCH v2] Add inline growth bias param

2019-11-27 Thread Jan Hubicka
> > 2. The results that it produced were exactly the same as those using the > > param uninlined-function-insns, so it would probably be redundant to add > > the additional parameter implemented this way. > > > > The implementation I tried is at: > >

Re: [PATCH v2] Add inline growth bias param

2019-11-27 Thread Richard Biener
On Mon, Nov 25, 2019 at 1:44 PM Graham Markall wrote: > > Hi Richard, > > Many thanks for the suggestion of an alternative implementation. I tried > implementing the suggestion, and I had a couple of observations: > > 1. As well as applying the bias in compute_fn_summary, it seemed to also > be

[PATCH v2] Add inline growth bias param

2019-11-25 Thread Graham Markall
Hi Richard, Many thanks for the suggestion of an alternative implementation. I tried implementing the suggestion, and I had a couple of observations: 1. As well as applying the bias in compute_fn_summary, it seemed to also be necessary to apply it in ip_update_overall_fn_summary to avoid an ICE