RE: [Patch] Fix PR53397

2012-10-10 Thread Uros Bizjak
Hello! >> +2012-10-09 Venkataramanan Kumar >> + >> +PR testsuite/53397 >> +* gcc/testsuite/gcc.dg: Remove test cases pr53397-1.c and pr53397-2.c >> +they are moved to gcc.target/i386 as they are specific to i386/x86_64. >> +* gcc/testsuite/gcc.target/i386/pr53397-1.c: New test >

Re: [Patch] Fix PR53397

2012-10-10 Thread Jakub Jelinek
On Wed, Oct 10, 2012 at 09:48:02AM +, Kumar, Venkataramanan wrote: > +2012-10-09 Venkataramanan Kumar > + > + PR testsuite/53397 > + * gcc/testsuite/gcc.dg: Remove test cases pr53397-1.c and pr53397-2.c > + they are moved to gcc.target/i386 as they are specific to i386/x86_64. >

RE: [Patch] Fix PR53397

2012-10-10 Thread Kumar, Venkataramanan
Wednesday, October 10, 2012 12:06 PM To: Kumar, Venkataramanan Cc: Richard Guenther; gcc-patches@gcc.gnu.org Subject: Re: [Patch] Fix PR53397 Hi! On Mon, Oct 08, 2012 at 10:01:53AM +, Kumar, Venkataramanan wrote: Both the testcases fail on i686-linux unfortunately: Excess errors: /usr/src/gcc/gcc

Re: [Patch] Fix PR53397

2012-10-09 Thread Jakub Jelinek
Hi! On Mon, Oct 08, 2012 at 10:01:53AM +, Kumar, Venkataramanan wrote: Both the testcases fail on i686-linux unfortunately: Excess errors: /usr/src/gcc/gcc/testsuite/gcc.dg/pr53397-1.c:1:0: warning: -fprefetch-loop-arrays not supported for this target (try -march switches) [enabled by defau

Re: [Patch] Fix PR53397

2012-10-08 Thread Richard Guenther
heck -k" passes with x86_64-unknown-linux-gnu. I presume also bootstrapped. Ok. Thanks, Richard. > Regards, > Venkat. > > -Original Message- > From: Richard Guenther [mailto:richard.guent...@gmail.com] > Sent: Thursday, October 04, 2012 6:26 PM > To: Kumar, Venkata

RE: [Patch] Fix PR53397

2012-10-08 Thread Kumar, Venkataramanan
richard.guent...@gmail.com] Sent: Thursday, October 04, 2012 6:26 PM To: Kumar, Venkataramanan Cc: Richard Guenther; gcc-patches@gcc.gnu.org Subject: Re: [Patch] Fix PR53397 On Tue, Oct 2, 2012 at 6:40 PM, Kumar, Venkataramanan wrote: > Hi Richi, > > (Snip) >> + (!cst_and_fits_

Re: [Patch] Fix PR53397

2012-10-04 Thread Richard Guenther
e place? Yes, call dump_mem_ref then, instead of repeating parts of its body. Richard. > Regards, > Venkat. > > > > -----Original Message- > From: Richard Guenther [mailto:rguent...@suse.de] > Sent: Tuesday, October 02, 2012 5:42 PM > To: Kumar, Venkataramanan > C

RE: [Patch] Fix PR53397

2012-10-02 Thread Kumar, Venkataramanan
ences. so I thought of dumping the references here. Is there a cleaner way to dump the references at one place? Regards, Venkat. -Original Message- From: Richard Guenther [mailto:rguent...@suse.de] Sent: Tuesday, October 02, 2012 5:42 PM To: Kumar, Venkataramanan Cc: gcc-patches

Re: [Patch] Fix PR53397

2012-10-02 Thread Richard Guenther
On Mon, 1 Oct 2012, venkataramanan.ku...@amd.com wrote: > Hi, > > The below patch fixes the FFT/Scimark regression caused by useless prefetch > generation. > > This fix tries to make prefetch less aggressive by prefetching arrays in the > inner loop, when the step is invariant in the entire loo

[Patch] Fix PR53397

2012-10-01 Thread venkataramanan.kumar
Hi, The below patch fixes the FFT/Scimark regression caused by useless prefetch generation. This fix tries to make prefetch less aggressive by prefetching arrays in the inner loop, when the step is invariant in the entire loop nest. GCC currently tries to prefetch invariant steps when they are