Re: [Patchv2 3/4] Control SRA and IPA-SRA by a param rather than MOVE_RATIO

2014-10-31 Thread Richard Biener
On Wed, Oct 29, 2014 at 3:39 PM, James Greenhalgh james.greenha...@arm.com wrote: On Wed, Oct 01, 2014 at 05:38:12PM +0100, James Greenhalgh wrote: On Fri, Sep 26, 2014 at 10:11:13AM +0100, Richard Biener wrote: On Thu, Sep 25, 2014 at 4:57 PM, James Greenhalgh james.greenha...@arm.com

Re: [Patchv2 3/4] Control SRA and IPA-SRA by a param rather than MOVE_RATIO

2014-10-29 Thread James Greenhalgh
On Wed, Oct 01, 2014 at 05:38:12PM +0100, James Greenhalgh wrote: On Fri, Sep 26, 2014 at 10:11:13AM +0100, Richard Biener wrote: On Thu, Sep 25, 2014 at 4:57 PM, James Greenhalgh james.greenha...@arm.com wrote: Given the special value to note the default for the new --params is zero a

Re: [Patchv2 3/4] Control SRA and IPA-SRA by a param rather than MOVE_RATIO

2014-10-01 Thread James Greenhalgh
On Fri, Sep 26, 2014 at 10:11:13AM +0100, Richard Biener wrote: On Thu, Sep 25, 2014 at 4:57 PM, James Greenhalgh james.greenha...@arm.com wrote: Given the special value to note the default for the new --params is zero a user cannot disable scalarization that way. I still somehow dislike

Re: [Patchv2 3/4] Control SRA and IPA-SRA by a param rather than MOVE_RATIO

2014-09-26 Thread Richard Biener
On Thu, Sep 25, 2014 at 4:57 PM, James Greenhalgh james.greenha...@arm.com wrote: Hi, After hookizing MOVE_BY_PIECES_P and migrating tree-inline.c, we are left with only one user of MOVE_RATIO - deciding the maximum size of aggregate for SRA. Past discussions have made it clear [1] that

[Patchv2 3/4] Control SRA and IPA-SRA by a param rather than MOVE_RATIO

2014-09-25 Thread James Greenhalgh
Hi, After hookizing MOVE_BY_PIECES_P and migrating tree-inline.c, we are left with only one user of MOVE_RATIO - deciding the maximum size of aggregate for SRA. Past discussions have made it clear [1] that keeping this use of MOVE_RATIO is undesirable. Clearly it is now also misnamed. The