Re: [PATCH] Correct cost model for strided loads

2012-06-12 Thread William J. Schmidt
On Tue, 2012-06-12 at 12:59 +0200, Richard Guenther wrote: > Btw, with PR53533 I now have a case where multiplications of v4si are > really expensive on x86 without SSE 4.1. But we only have vect_stmt_cost > and no further subdivision ... > > Thus we'd need a tree_code argument to the cost hook.

Re: [PATCH] Correct cost model for strided loads

2012-06-12 Thread Richard Guenther
On Mon, Jun 11, 2012 at 8:37 PM, Richard Guenther wrote: > On Mon, Jun 11, 2012 at 5:01 PM, William J. Schmidt > wrote: >> >> >> On Mon, 2012-06-11 at 16:10 +0200, Richard Guenther wrote: >>> On Mon, 11 Jun 2012, William J. Schmidt wrote: >>> >>> > >>> > >>> > On Mon, 2012-06-11 at 11:15 +0200, R

Re: [PATCH] Correct cost model for strided loads

2012-06-11 Thread Richard Guenther
On Mon, Jun 11, 2012 at 5:01 PM, William J. Schmidt wrote: > > > On Mon, 2012-06-11 at 16:10 +0200, Richard Guenther wrote: >> On Mon, 11 Jun 2012, William J. Schmidt wrote: >> >> > >> > >> > On Mon, 2012-06-11 at 11:15 +0200, Richard Guenther wrote: >> > > On Sun, Jun 10, 2012 at 5:58 PM, William

Re: [PATCH] Correct cost model for strided loads

2012-06-11 Thread William J. Schmidt
On Mon, 2012-06-11 at 16:10 +0200, Richard Guenther wrote: > On Mon, 11 Jun 2012, William J. Schmidt wrote: > > > > > > > On Mon, 2012-06-11 at 11:15 +0200, Richard Guenther wrote: > > > On Sun, Jun 10, 2012 at 5:58 PM, William J. Schmidt > > > wrote: > > > > The fix for PR53331 caused a degr

Re: [PATCH] Correct cost model for strided loads

2012-06-11 Thread Richard Guenther
On Mon, 11 Jun 2012, William J. Schmidt wrote: > > > On Mon, 2012-06-11 at 11:15 +0200, Richard Guenther wrote: > > On Sun, Jun 10, 2012 at 5:58 PM, William J. Schmidt > > wrote: > > > The fix for PR53331 caused a degradation to 187.facerec on > > > powerpc64-unknown-linux-gnu. The following s

Re: [PATCH] Correct cost model for strided loads

2012-06-11 Thread William J. Schmidt
On Mon, 2012-06-11 at 11:15 +0200, Richard Guenther wrote: > On Sun, Jun 10, 2012 at 5:58 PM, William J. Schmidt > wrote: > > The fix for PR53331 caused a degradation to 187.facerec on > > powerpc64-unknown-linux-gnu. The following simple patch reverses the > > degradation without otherwise aff

Re: [PATCH] Correct cost model for strided loads

2012-06-11 Thread Richard Guenther
On Sun, Jun 10, 2012 at 5:58 PM, William J. Schmidt wrote: > The fix for PR53331 caused a degradation to 187.facerec on > powerpc64-unknown-linux-gnu.  The following simple patch reverses the > degradation without otherwise affecting SPEC cpu2000 or cpu2006. > Bootstrapped and regtested on that pl

[PATCH] Correct cost model for strided loads

2012-06-10 Thread William J. Schmidt
The fix for PR53331 caused a degradation to 187.facerec on powerpc64-unknown-linux-gnu. The following simple patch reverses the degradation without otherwise affecting SPEC cpu2000 or cpu2006. Bootstrapped and regtested on that platform with no new regressions. Ok for trunk? Thanks, Bill 2012-