Re: [PATCH, i386] Enable -mprefer-avx128 by default for Bulldozer

2011-06-29 Thread Jakub Jelinek
On Wed, Jun 29, 2011 at 09:49:52AM +0200, Jan Hubicka wrote: > > * config/i386/i386.opt (mprefer-avx128): Redefine the flag as a Mask > > option. > > > > * config/i386/i386.h (ix86_tune_indices): Add X86_TUNE_AVX128_OPTIMAL > > entry. > > (TARGET_AVX128_OPTIMAL): New definition. > >

Re: [PATCH, i386] Enable -mprefer-avx128 by default for Bulldozer

2011-06-29 Thread Jan Hubicka
> * config/i386/i386.opt (mprefer-avx128): Redefine the flag as a Mask > option. > > * config/i386/i386.h (ix86_tune_indices): Add X86_TUNE_AVX128_OPTIMAL > entry. > (TARGET_AVX128_OPTIMAL): New definition. > > * config/i386/i386.c (initial_ix86_tune_features): Initializ

RE: [PATCH, i386] Enable -mprefer-avx128 by default for Bulldozer

2011-06-28 Thread Fang, Changpeng
-patches@gcc.gnu.org; rguent...@suse.de Subject: RE: [PATCH, i386] Enable -mprefer-avx128 by default for Bulldozer Is this patch OK to commit to trunk? Also I would like to backport this patch to gcc 4.6 branch. Do I have to send a separate request or use this one? Thanks, Changpeng

RE: [PATCH, i386] Enable -mprefer-avx128 by default for Bulldozer

2011-06-27 Thread Fang, Changpeng
: Uros Bizjak; gcc-patches@gcc.gnu.org; rguent...@suse.de Subject: RE: [PATCH, i386] Enable -mprefer-avx128 by default for Bulldozer Hi, I have no preference in tune feature coding. But I agree with you it's better to put similar things together. I modified the code following your suggestion

RE: [PATCH, i386] Enable -mprefer-avx128 by default for Bulldozer

2011-06-24 Thread Fang, Changpeng
[hubi...@ucw.cz] Sent: Thursday, June 23, 2011 6:20 PM To: Fang, Changpeng Cc: Uros Bizjak; gcc-patches@gcc.gnu.org; hubi...@ucw.cz; rguent...@suse.de Subject: Re: [PATCH, i386] Enable -mprefer-avx128 by default for Bulldozer Hi, > --- a/gcc/config/i386/i386.c > +++ b/gcc/config/i386/i386.c &

Re: [PATCH, i386] Enable -mprefer-avx128 by default for Bulldozer

2011-06-23 Thread Jan Hubicka
Hi, > --- a/gcc/config/i386/i386.c > +++ b/gcc/config/i386/i386.c > @@ -2128,6 +2128,9 @@ static const unsigned int > x86_avx256_split_unaligned_load > static const unsigned int x86_avx256_split_unaligned_store >= m_COREI7 | m_BDVER1 | m_GENERIC; > > +static const unsigned int x86_prefer_av

Re: [PATCH, i386] Enable -mprefer-avx128 by default for Bulldozer

2011-06-23 Thread Jakub Jelinek
On Thu, Jun 23, 2011 at 03:41:01PM -0500, Fang, Changpeng wrote: > This patch enables 128-bit avx instruction generation for the auto-vectorizer > for AMD bulldozer > machines. This enablement gives additional ~3% improvement on polyhedron 2005 > and cpu2006 > floating point programs. > > The p

[PATCH, i386] Enable -mprefer-avx128 by default for Bulldozer

2011-06-23 Thread Fang, Changpeng
Hi, This patch enables 128-bit avx instruction generation for the auto-vectorizer for AMD bulldozer machines. This enablement gives additional ~3% improvement on polyhedron 2005 and cpu2006 floating point programs. The patch passed bootstrapping on a x86_64-unknown-linux-gnu system with Bulld