Re: [PATCH 2/3] Incorporate aggregate jump functions into inlining analysis

2012-08-29 Thread H.J. Lu
On Thu, Aug 2, 2012 at 12:28 PM, Martin Jambor mjam...@suse.cz wrote: Hi, this patch uses the aggregate jump functions created by the previous patch in the series to determine benefits of inlining a particular call graph edge. It has not changed much since the last time I posted it, except

Re: [PATCH 2/3] Incorporate aggregate jump functions into inlining analysis

2012-08-15 Thread Martin Jambor
Hi, On Fri, Aug 10, 2012 at 05:12:31AM +0200, Jan Hubicka wrote: Do you have any data on memory usage? I was originally concerned about memory use of the whole predicate thingy on WPA level. Eventually we could add simple inheritance on conditions and sort them into mutiple vectors if

Re: [PATCH 2/3] Incorporate aggregate jump functions into inlining analysis

2012-08-11 Thread Martin Jambor
Hi, On Fri, Aug 10, 2012 at 04:39:44PM +0200, Martin Jambor wrote: On Fri, Aug 10, 2012 at 05:12:31AM +0200, Jan Hubicka wrote: Hi, ... 2012-07-31 Martin Jambor mjam...@suse.cz PR fortran/48636 * ipa-inline.h (condition): New fields offset, agg_contents and

Re: [PATCH 2/3] Incorporate aggregate jump functions into inlining analysis

2012-08-11 Thread Jan Hubicka
This is the patch I have committed after resolving a conflict (and another round of bootstrapping and testing). The ChangeLog is still the same. The previous version of the patch (but with tree code already converted to an enum), the aggregate jump functions together with enlarged

Re: [PATCH 2/3] Incorporate aggregate jump functions into inlining analysis

2012-08-09 Thread Jan Hubicka
Hi, this patch uses the aggregate jump functions created by the previous patch in the series to determine benefits of inlining a particular call graph edge. It has not changed much since the last time I posted it, except for the presence of by_ref flags and removal of checks required by

Re: [PATCH 2/3] Incorporate aggregate jump functions into inlining analysis

2012-08-03 Thread Martin Jambor
On Thu, Aug 02, 2012 at 09:28:11PM +0200, Martin Jambor wrote: Hi, this patch uses the aggregate jump functions created by the previous patch in the series to determine benefits of inlining a particular call graph edge. It has not changed much since the last time I posted it, except for

[PATCH 2/3] Incorporate aggregate jump functions into inlining analysis

2012-08-02 Thread Martin Jambor
Hi, this patch uses the aggregate jump functions created by the previous patch in the series to determine benefits of inlining a particular call graph edge. It has not changed much since the last time I posted it, except for the presence of by_ref flags and removal of checks required by TBAA

[PATCH 2/3] Incorporate aggregate jump functions into inlining analysis

2012-07-02 Thread Martin Jambor
Hi, this patch uses the aggregate jump functions created by a previous patch to determine benefits of inlining a particular call graph edge. It does so by fairly straightforward way. It a flag to struct condition to specify it is actually an aggregate value at an offset, also newly stored in