Re: [PATCH] Add flag to control straight-line strength reduction

2012-07-18 Thread Richard Guenther
On Wed, 18 Jul 2012, Steven Bosscher wrote: > On Wed, Jul 18, 2012 at 3:24 PM, William J. Schmidt > wrote: > > It turns out I was looking at a very old copy of the manual, and the > > -ftree... stuff is not as prevalent now as it once was. I'll just go > > with -fslsr to be consistent with -fgcs

Re: [PATCH] Add flag to control straight-line strength reduction

2012-07-18 Thread Richard Guenther
On Wed, 18 Jul 2012, William J. Schmidt wrote: > Here's the patch with documentation changes included. I also cleaned up > missing work from a couple of my previous patches, so > -fhoist-adjacent-loads is documented now, and -fvect-cost-model is added > to the list of options on by default at -O3

Re: [PATCH] Add flag to control straight-line strength reduction

2012-07-18 Thread Steven Bosscher
On Wed, Jul 18, 2012 at 3:24 PM, William J. Schmidt wrote: > It turns out I was looking at a very old copy of the manual, and the > -ftree... stuff is not as prevalent now as it once was. I'll just go > with -fslsr to be consistent with -fgcse, -fipa-sra, etc. Sadly, it is more prevalent than it

Re: [PATCH] Add flag to control straight-line strength reduction

2012-07-18 Thread William J. Schmidt
Here's the patch with documentation changes included. I also cleaned up missing work from a couple of my previous patches, so -fhoist-adjacent-loads is documented now, and -fvect-cost-model is added to the list of options on by default at -O3. Ok for trunk? Thanks, Bill 2012-07-18 Bill Schmid

Re: [PATCH] Add flag to control straight-line strength reduction

2012-07-18 Thread William J. Schmidt
On Wed, 2012-07-18 at 08:24 -0500, William J. Schmidt wrote: > On Wed, 2012-07-18 at 11:01 +0200, Richard Guenther wrote: > > On Wed, 18 Jul 2012, Steven Bosscher wrote: > > > > > On Wed, Jul 18, 2012 at 9:59 AM, Richard Guenther > > > wrote: > > > > On Tue, 17 Jul 2012, William J. Schmidt wrote

Re: [PATCH] Add flag to control straight-line strength reduction

2012-07-18 Thread William J. Schmidt
On Wed, 2012-07-18 at 11:01 +0200, Richard Guenther wrote: > On Wed, 18 Jul 2012, Steven Bosscher wrote: > > > On Wed, Jul 18, 2012 at 9:59 AM, Richard Guenther wrote: > > > On Tue, 17 Jul 2012, William J. Schmidt wrote: > > > > > >> I overlooked adding a pass-control flag for strength reduction,

Re: [PATCH] Add flag to control straight-line strength reduction

2012-07-18 Thread Eric Botcazou
> In the past, -fstrength-reduce applied to loop strength reduction in > loop.c. I don't think it should be re-used for a completely different > code transformation. Seconded. -- Eric Botcazou

Re: [PATCH] Add flag to control straight-line strength reduction

2012-07-18 Thread Richard Guenther
On Wed, 18 Jul 2012, Steven Bosscher wrote: > On Wed, Jul 18, 2012 at 9:59 AM, Richard Guenther wrote: > > On Tue, 17 Jul 2012, William J. Schmidt wrote: > > > >> I overlooked adding a pass-control flag for strength reduction, added > >> here. I named it -ftree-slsr for consistency with other -f

Re: [PATCH] Add flag to control straight-line strength reduction

2012-07-18 Thread Steven Bosscher
On Wed, Jul 18, 2012 at 9:59 AM, Richard Guenther wrote: > On Tue, 17 Jul 2012, William J. Schmidt wrote: > >> I overlooked adding a pass-control flag for strength reduction, added >> here. I named it -ftree-slsr for consistency with other -ftree- flags, >> but could change it to -fgimple-slsr if

Re: [PATCH] Add flag to control straight-line strength reduction

2012-07-18 Thread Richard Guenther
On Tue, 17 Jul 2012, William J. Schmidt wrote: > I overlooked adding a pass-control flag for strength reduction, added > here. I named it -ftree-slsr for consistency with other -ftree- flags, > but could change it to -fgimple-slsr if you prefer that for a pass named > gimple-ssa-... > > Bootstra

[PATCH] Add flag to control straight-line strength reduction

2012-07-17 Thread William J. Schmidt
I overlooked adding a pass-control flag for strength reduction, added here. I named it -ftree-slsr for consistency with other -ftree- flags, but could change it to -fgimple-slsr if you prefer that for a pass named gimple-ssa-... Bootstrapped and tested on powerpc-unknown-linux-gnu with no new reg