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