Re: [PATCH] Strength reduction part 3 of 4: candidates with unknown strides

2012-08-09 Thread William J. Schmidt
On Wed, 2012-08-08 at 19:22 -0700, Janis Johnson wrote: > On 08/08/2012 06:41 PM, William J. Schmidt wrote: > > On Wed, 2012-08-08 at 15:35 -0700, Janis Johnson wrote: > >> On 08/08/2012 03:27 PM, Andrew Pinski wrote: > >>> On Wed, Aug 8, 2012 at 3:25 PM, H.J. Lu wrote: > On Wed, Aug 1, 2012

Re: [PATCH] Strength reduction part 3 of 4: candidates with unknown strides

2012-08-09 Thread Richard Guenther
On Wed, 8 Aug 2012, H.J. Lu wrote: > On Wed, Aug 1, 2012 at 10:36 AM, William J. Schmidt > wrote: > > Greetings, > > > > Thanks for the review of part 2! Here's another chunk of the SLSR code > > (I feel I owe you a few beers at this point). This performs analysis > > and replacement on groups

Re: [PATCH] Strength reduction part 3 of 4: candidates with unknown strides

2012-08-08 Thread Janis Johnson
On 08/08/2012 06:41 PM, William J. Schmidt wrote: > On Wed, 2012-08-08 at 15:35 -0700, Janis Johnson wrote: >> On 08/08/2012 03:27 PM, Andrew Pinski wrote: >>> On Wed, Aug 8, 2012 at 3:25 PM, H.J. Lu wrote: On Wed, Aug 1, 2012 at 10:36 AM, William J. Schmidt wrote: > +/* { dg-do co

Re: [PATCH] Strength reduction part 3 of 4: candidates with unknown strides

2012-08-08 Thread William J. Schmidt
On Wed, 2012-08-08 at 15:35 -0700, Janis Johnson wrote: > On 08/08/2012 03:27 PM, Andrew Pinski wrote: > > On Wed, Aug 8, 2012 at 3:25 PM, H.J. Lu wrote: > >> On Wed, Aug 1, 2012 at 10:36 AM, William J. Schmidt > >> wrote: > >>> Greetings, > >>> > >>> Thanks for the review of part 2! Here's anot

Re: [PATCH] Strength reduction part 3 of 4: candidates with unknown strides

2012-08-08 Thread Janis Johnson
On 08/08/2012 03:27 PM, Andrew Pinski wrote: > On Wed, Aug 8, 2012 at 3:25 PM, H.J. Lu wrote: >> On Wed, Aug 1, 2012 at 10:36 AM, William J. Schmidt >> wrote: >>> Greetings, >>> >>> Thanks for the review of part 2! Here's another chunk of the SLSR code >>> (I feel I owe you a few beers at this p

Re: [PATCH] Strength reduction part 3 of 4: candidates with unknown strides

2012-08-08 Thread Mike Stump
On Aug 8, 2012, at 3:25 PM, H.J. Lu wrote: > This doesn't work on x32 nor Linux/ia32 since -m32 > may not be needed for ILP32. This patch works for > me. OK to install? Ok.

Re: [PATCH] Strength reduction part 3 of 4: candidates with unknown strides

2012-08-08 Thread Andrew Pinski
On Wed, Aug 8, 2012 at 3:25 PM, H.J. Lu wrote: > On Wed, Aug 1, 2012 at 10:36 AM, William J. Schmidt > wrote: >> Greetings, >> >> Thanks for the review of part 2! Here's another chunk of the SLSR code >> (I feel I owe you a few beers at this point). This performs analysis >> and replacement on

Re: [PATCH] Strength reduction part 3 of 4: candidates with unknown strides

2012-08-08 Thread H.J. Lu
On Wed, Aug 1, 2012 at 10:36 AM, William J. Schmidt wrote: > Greetings, > > Thanks for the review of part 2! Here's another chunk of the SLSR code > (I feel I owe you a few beers at this point). This performs analysis > and replacement on groups of related candidates having an SSA name > (rather

[PATCH] Strength reduction part 3 of 4: candidates with unknown strides

2012-08-01 Thread William J. Schmidt
Greetings, Thanks for the review of part 2! Here's another chunk of the SLSR code (I feel I owe you a few beers at this point). This performs analysis and replacement on groups of related candidates having an SSA name (rather than a constant) for a stride. This leaves only the conditional incre

Re: [PATCH] Strength reduction

2012-06-26 Thread Richard Guenther
On Tue, 26 Jun 2012, William J. Schmidt wrote: > On Tue, 2012-06-26 at 15:06 +0200, Richard Guenther wrote: > > On Mon, 25 Jun 2012, William J. Schmidt wrote: > > > > > Here's a new version of the main strength reduction patch, addressing > > > previous comments. A couple of quick notes: > > >

[PATCH] Strength reduction

2012-06-25 Thread William J. Schmidt
Here's a new version of the main strength reduction patch, addressing previous comments. A couple of quick notes: * I opened PR53773 and PR53774 for the cases where commutative operations were encountered with a constant in rhs1. This version of the patch still has the gcc_asserts in place to ca

Re: [PATCH] Strength reduction preliminaries

2012-06-22 Thread William J. Schmidt
On Fri, 2012-06-22 at 10:44 +0200, Richard Guenther wrote: > On Thu, 21 Jun 2012, William J. Schmidt wrote: > > I ran into a glitch with multiply_by_const_cost. The original code > > declared a static htab_t in the function and allocated it on demand. > > When I tried adding a second one in the s

Re: [PATCH] Strength reduction preliminaries

2012-06-22 Thread William J. Schmidt
On Fri, 2012-06-22 at 10:44 +0200, Richard Guenther wrote: > On Thu, 21 Jun 2012, William J. Schmidt wrote: > > > As promised, this breaks out the changes to the IVOPTS cost model and > > the added function in double-int.c. Please let me know if you would > > rather see me attempt to consolidate

Re: [PATCH] Strength reduction preliminaries

2012-06-22 Thread Richard Guenther
On Thu, 21 Jun 2012, William J. Schmidt wrote: > As promised, this breaks out the changes to the IVOPTS cost model and > the added function in double-int.c. Please let me know if you would > rather see me attempt to consolidate the IVOPTS logic into expmed.c per > Richard H's suggestion. If we s

[PATCH] Strength reduction preliminaries

2012-06-21 Thread William J. Schmidt
As promised, this breaks out the changes to the IVOPTS cost model and the added function in double-int.c. Please let me know if you would rather see me attempt to consolidate the IVOPTS logic into expmed.c per Richard H's suggestion. I ran into a glitch with multiply_by_const_cost. The original