RE: [PATCH, i386]: Fix PR51109, symbol size in scheduler state machine is reduced

2012-10-10 Thread Gopalasubramanian, Ganesh
...@gcc.gnu.org Subject: Re: [PATCH, i386]: Fix PR51109, symbol size in scheduler state machine is reduced On 10/10/2012 01:00 PM, Jakub Jelinek wrote: > I have removed the extra line as obvious in SVN, to allow my > bootstraps to continue. Thanks! Paolo.

Re: [PATCH, i386]: Fix PR51109, symbol size in scheduler state machine is reduced

2012-10-10 Thread Paolo Carlini
On 10/10/2012 01:00 PM, Jakub Jelinek wrote: I have removed the extra line as obvious in SVN, to allow my bootstraps to continue. Thanks! Paolo.

Re: [PATCH, i386]: Fix PR51109, symbol size in scheduler state machine is reduced

2012-10-10 Thread Jakub Jelinek
On Wed, Oct 10, 2012 at 12:57:09PM +0200, Paolo Carlini wrote: > On 10/10/2012 12:53 PM, Jakub Jelinek wrote: > >Yeah, clearly a different version of the patch has been posted > >vs. what has been checked in. The difference is removal of the > >(define_cpu_unit "bdver1-mult" "bdver1_mult") > >line

Re: [PATCH, i386]: Fix PR51109, symbol size in scheduler state machine is reduced

2012-10-10 Thread Paolo Carlini
Hi, On 10/10/2012 12:53 PM, Jakub Jelinek wrote: Yeah, clearly a different version of the patch has been posted vs. what has been checked in. The difference is removal of the (define_cpu_unit "bdver1-mult" "bdver1_mult") line (present in the posted patch, not in the checked in patch). Also, th

Re: [PATCH, i386]: Fix PR51109, symbol size in scheduler state machine is reduced

2012-10-10 Thread Jakub Jelinek
On Wed, Oct 10, 2012 at 12:29:04PM +0200, Paolo Carlini wrote: > Hi, > > On 10/03/2012 11:57 AM, Gopalasubramanian, Ganesh wrote: > >Testing was done before posting the patch. It was successful. > This change is now in: > > http://gcc.gnu.org/ml/gcc-cvs/2012-10/msg00418.html > > and it looks

Re: [PATCH, i386]: Fix PR51109, symbol size in scheduler state machine is reduced

2012-10-10 Thread Paolo Carlini
Hi, On 10/03/2012 11:57 AM, Gopalasubramanian, Ganesh wrote: Testing was done before posting the patch. It was successful. This change is now in: http://gcc.gnu.org/ml/gcc-cvs/2012-10/msg00418.html and it looks like is breaking the build for me: build/genattrtab /scratch/Gcc/svn-dir

RE: [PATCH, i386]: Fix PR51109, symbol size in scheduler state machine is reduced

2012-10-03 Thread Gopalasubramanian, Ganesh
, symbol size in scheduler state machine is reduced On Thu, Sep 27, 2012 at 10:30 AM, Gopalasubramanian, Ganesh wrote: > This is a fix for PR 51109. > > There are three changes > > 1. Microcoded instructions are considered as single issue instructions > and are therefore is

Re: [PATCH, i386]: Fix PR51109, symbol size in scheduler state machine is reduced

2012-09-27 Thread Uros Bizjak
On Thu, Sep 27, 2012 at 10:30 AM, Gopalasubramanian, Ganesh wrote: > This is a fix for PR 51109. > > There are three changes > > 1. Microcoded instructions are considered as single issue instructions > and are therefore issued to a separate execution unit. > 2. The multiplier unit is a