Re: [PATCH PR50572] Tune loop alignment for Atom

2011-10-20 Thread H.J. Lu
On Thu, Oct 20, 2011 at 1:05 AM, Sergey Ostanevich wrote: >> Please provide a patch which can be applied.  Cut/paste doesn't create >> a working patch.  Please attach it. >> >> -- >> H.J. >> > > Will that works? > Sergos. > > diff --git a/gcc/ChangeLog b/gcc/ChangeLog > index 6c73404..e21cf86 1006

Re: [PATCH PR50572] Tune loop alignment for Atom

2011-10-20 Thread Sergey Ostanevich
> Please provide a patch which can be applied.  Cut/paste doesn't create > a working patch.  Please attach it. > > -- > H.J. > Will that works? Sergos. diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 6c73404..e21cf86 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2011-10-20 Ser

Re: [PATCH PR50572] Tune loop alignment for Atom

2011-10-19 Thread H.J. Lu
On Wed, Oct 19, 2011 at 9:30 AM, Sergey Ostanevich wrote: > 2011-10-18  Sergey Ostanevich > >    * config/i386/i386.c (processor_target_table): Change Atom >    align_loops_max_skip to 15. Please add PR # in ChangeLog. > > diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c > index 2c

Re: [PATCH PR50572] Tune loop alignment for Atom

2011-10-19 Thread H.J. Lu
On Wed, Oct 19, 2011 at 6:09 AM, Sergey Ostanevich wrote: > On Wed, Oct 19, 2011 at 4:46 PM, Uros Bizjak wrote: >> On Wed, Oct 19, 2011 at 2:26 PM, Sergey Ostanevich >> wrote: >> You can just change the default in processor_target_table. Uros. >>> >>> Will it be applicable d

Re: [PATCH PR50572] Tune loop alignment for Atom

2011-10-19 Thread Sergey Ostanevich
On Wed, Oct 19, 2011 at 4:46 PM, Uros Bizjak wrote: > On Wed, Oct 19, 2011 at 2:26 PM, Sergey Ostanevich > wrote: > >>> You can just change the default in processor_target_table. >>> >>> Uros. >>> >> >> Will it be applicable during optimizations for size? It will hurt, >> although not much (see

Re: [PATCH PR50572] Tune loop alignment for Atom

2011-10-19 Thread Uros Bizjak
On Wed, Oct 19, 2011 at 2:26 PM, Sergey Ostanevich wrote: >> You can just change the default in processor_target_table. >> >> Uros. >> > > Will it be applicable during optimizations for size? It will hurt, > although not much (see PR). Looking at the code, I'd say that we don't handle -Os in dif

Re: [PATCH PR50572] Tune loop alignment for Atom

2011-10-19 Thread Sergey Ostanevich
On Wed, Oct 19, 2011 at 4:14 PM, Uros Bizjak wrote: > > You can just change the default in processor_target_table. > > Uros. > Will it be applicable during optimizations for size? It will hurt, although not much (see PR). New patch is below. Ok for trunk as obvious? Sergos 2011-10-19 Sergey Os

Re: [PATCH PR50572] Tune loop alignment for Atom

2011-10-19 Thread Uros Bizjak
Hello! > 2011-10-19 Sergey Ostanevich > > * gcc/config/i386/i386.c (ix86_option_override_internal): use loop > align by 16 bytes for Atom platform > > diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c > index 2c53423..7a93144 100644 > --- a/gcc/config/i386/i386.c > +++ b/gcc/co

[PATCH PR50572] Tune loop alignment for Atom

2011-10-19 Thread Sergey Ostanevich
Hi! Here is a patch to address PR50572. Successfully bootstrapped and passed make check on x86_64-linux. regards, Sergos 2011-10-19 Sergey Ostanevich * gcc/config/i386/i386.c (ix86_option_override_internal): use loop align by 16 bytes for Atom platform diff --git a/gcc/config/i386