This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, i386]: Fix PR51109, symbol size in scheduler state machine is reduced
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: "Gopalasubramanian, Ganesh" <Ganesh dot Gopalasubramanian at amd dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 27 Sep 2012 14:27:04 +0200
- Subject: Re: [PATCH, i386]: Fix PR51109, symbol size in scheduler state machine is reduced
- References: <EB4625145972F94C9680D8CADD6516152A36BE71@sausexdag04.amd.com>
On Thu, Sep 27, 2012 at 10:30 AM, Gopalasubramanian, Ganesh
<Ganesh.Gopalasubramanian@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. The multiplier unit is attached to execution unit 1 (ieu1). Since ieu is handled as a separate automaton in the patch, separate "mult" automaton is not required.
> 3. The integer execution units (2AGUs and 2EXs) are now decoupled. Now, they are described as separate automatons.
>
> Is it OK for upstream?
>
> Regards
> Ganesh
>
> 2012-09-27 Ganesh Gopalasubramanian <Ganesh.Gopalasubramanian@amd.com>
>
> PR 51109
> * gcc/config/i386/bdver1.md (bdver1_int): Automaton has been
> split to reduce state transitions.
OK for mainline, if tested according to [1].
[1] http://gcc.gnu.org/contribute.html#testing
Thanks,
Uros.