Re: [PATCH][RFC] Move IVOPTs closer to RTL expansion

2013-09-12 Thread Richard Biener
On Wed, 11 Sep 2013, Pat Haugen wrote: > On 09/04/2013 04:20 AM, Richard Biener wrote: > > Any help with benchmarking this on targets other than x86_64 > > is appreciated (I'll re-do x86_64). > > > I ran CPU2000 and CPU2006 on PowerPC comparing the patch. CPU2000 had 3 > benchmarks degrade in the

Re: [PATCH][RFC] Move IVOPTs closer to RTL expansion

2013-09-11 Thread Pat Haugen
On 09/04/2013 04:20 AM, Richard Biener wrote: Any help with benchmarking this on targets other than x86_64 is appreciated (I'll re-do x86_64). I ran CPU2000 and CPU2006 on PowerPC comparing the patch. CPU2000 had 3 benchmarks degrade in the 4%-6%range (254.gap, 168.wupwise, 173.applu).CPU2006

Re: [PATCH][RFC] Move IVOPTs closer to RTL expansion

2013-09-10 Thread Richard Biener
On Tue, 10 Sep 2013, Jeff Law wrote: > On 09/10/2013 07:23 AM, Richard Biener wrote: > > > > update-ssa for example just queues to release SSA names. But yes, > > re-using SSA names from a freelist has issues and advantages ;) > Yup. I've wondered from time to time if we could do away with the

Re: [PATCH][RFC] Move IVOPTs closer to RTL expansion

2013-09-10 Thread Jeff Law
On 09/10/2013 07:23 AM, Richard Biener wrote: update-ssa for example just queues to release SSA names. But yes, re-using SSA names from a freelist has issues and advantages ;) Yup. I've wondered from time to time if we could do away with the name manager -- IIRCC it was written when we took e

Re: [PATCH][RFC] Move IVOPTs closer to RTL expansion

2013-09-10 Thread Richard Biener
On Tue, 10 Sep 2013, Jeff Law wrote: > On 09/10/2013 02:00 AM, Richard Biener wrote: > > On Mon, 9 Sep 2013, Steven Bosscher wrote: > > > > > On Mon, Sep 9, 2013 at 10:01 AM, Richard Biener wrote: > > > > > > > First, the loop passes that at the moment preceede IVOPTs leave > > > > > > > around I

Re: [PATCH][RFC] Move IVOPTs closer to RTL expansion

2013-09-10 Thread Jeff Law
On 09/10/2013 02:00 AM, Richard Biener wrote: On Mon, 9 Sep 2013, Steven Bosscher wrote: On Mon, Sep 9, 2013 at 10:01 AM, Richard Biener wrote: First, the loop passes that at the moment preceede IVOPTs leave around IL that is in desparate need of basic re-optimization like CSE, constant propag

Re: [PATCH][RFC] Move IVOPTs closer to RTL expansion

2013-09-10 Thread Richard Biener
On Mon, 9 Sep 2013, Steven Bosscher wrote: > On Mon, Sep 9, 2013 at 10:01 AM, Richard Biener wrote: > >> >> First, the loop passes that at the moment preceede IVOPTs leave > >> >> around IL that is in desparate need of basic re-optimization > >> >> like CSE, constant propagation and DCE. That put

Re: [PATCH][RFC] Move IVOPTs closer to RTL expansion

2013-09-10 Thread Bin.Cheng
On Tue, Sep 10, 2013 at 3:30 AM, Steven Bosscher wrote: > On Mon, Sep 9, 2013 at 10:01 AM, Richard Biener wrote: >>> >> First, the loop passes that at the moment preceede IVOPTs leave >>> >> around IL that is in desparate need of basic re-optimization >>> >> like CSE, constant propagation and DCE.

Re: [PATCH][RFC] Move IVOPTs closer to RTL expansion

2013-09-09 Thread Steven Bosscher
On Mon, Sep 9, 2013 at 10:01 AM, Richard Biener wrote: >> >> First, the loop passes that at the moment preceede IVOPTs leave >> >> around IL that is in desparate need of basic re-optimization >> >> like CSE, constant propagation and DCE. That puts extra load >> >> on IVOPTs and its cost model, inc

Re: [PATCH][RFC] Move IVOPTs closer to RTL expansion

2013-09-09 Thread Richard Biener
On Sun, 8 Sep 2013, pins...@gmail.com wrote: > On Sep 8, 2013, at 7:01 PM, "Bin.Cheng" wrote: > > > On Wed, Sep 4, 2013 at 5:20 PM, Richard Biener wrote: > >> > >> The patch below moves IVOPTs out of the GIMPLE loop pipeline more > >> closer to RTL expansion. That's done for multiple reasons.

Re: [PATCH][RFC] Move IVOPTs closer to RTL expansion

2013-09-08 Thread pinskia
On Sep 8, 2013, at 7:01 PM, "Bin.Cheng" wrote: > On Wed, Sep 4, 2013 at 5:20 PM, Richard Biener wrote: >> >> The patch below moves IVOPTs out of the GIMPLE loop pipeline more >> closer to RTL expansion. That's done for multiple reasons. >> >> First, the loop passes that at the moment preceede

Re: [PATCH][RFC] Move IVOPTs closer to RTL expansion

2013-09-08 Thread Bin.Cheng
On Mon, Sep 9, 2013 at 10:01 AM, Bin.Cheng wrote: > On Wed, Sep 4, 2013 at 5:20 PM, Richard Biener wrote: >> >> The patch below moves IVOPTs out of the GIMPLE loop pipeline more >> closer to RTL expansion. That's done for multiple reasons. >> >> First, the loop passes that at the moment preceede

Re: [PATCH][RFC] Move IVOPTs closer to RTL expansion

2013-09-08 Thread Bin.Cheng
On Wed, Sep 4, 2013 at 5:20 PM, Richard Biener wrote: > > The patch below moves IVOPTs out of the GIMPLE loop pipeline more > closer to RTL expansion. That's done for multiple reasons. > > First, the loop passes that at the moment preceede IVOPTs leave > around IL that is in desparate need of bas

[PATCH][RFC] Move IVOPTs closer to RTL expansion

2013-09-04 Thread Richard Biener
The patch below moves IVOPTs out of the GIMPLE loop pipeline more closer to RTL expansion. That's done for multiple reasons. First, the loop passes that at the moment preceede IVOPTs leave around IL that is in desparate need of basic re-optimization like CSE, constant propagation and DCE. That