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