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

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 like is

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, the

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 (present in

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-03 Thread Gopalasubramanian, Ganesh
, symbol size in scheduler state machine is reduced On Thu, Sep 27, 2012 at 10:30 AM, Gopalasubramanian, Ganesh ganesh.gopalasubraman...@amd.com wrote: This is a fix for PR 51109. There are three changes 1. Microcoded instructions are considered as single issue instructions

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

2012-09-27 Thread Gopalasubramanian, Ganesh
Hi All, 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 attached to execution unit 1 (ieu1). Since ieu is handled as a separate

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 ganesh.gopalasubraman...@amd.com 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.