This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][AArch64] Add missing fcsel in Cortex-A57 scheduler
- From: James Greenhalgh <james dot greenhalgh at arm dot com>
- To: Wilco Dijkstra <Wilco dot Dijkstra at arm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, nd <nd at arm dot com>
- Date: Thu, 2 Jun 2016 17:24:30 +0100
- Subject: Re: [PATCH][AArch64] Add missing fcsel in Cortex-A57 scheduler
- Authentication-results: sourceware.org; auth=none
- Nodisclaimer: True
- References: <DB3PR08MB008991A651ED2D80654D46FF83580 at DB3PR08MB0089 dot eurprd08 dot prod dot outlook dot com>
- Spamdiagnosticmetadata: NSPM
- Spamdiagnosticoutput: 1:23
On Thu, Jun 02, 2016 at 04:09:32PM +0000, Wilco Dijkstra wrote:
> The Cortex-A57 scheduler is missing fcsel, so add it.
>
> OK for commit?
OK.
Thanks,
James
>
> ChangeLog:
> 2016-06-02 Wilco Dijkstra <wdijkstr@arm.com>
>
> * config/arm/cortex-a57.md (cortex_a57_fp_cpys): Add fcsel.
>
> ---
> diff --git a/gcc/config/arm/cortex-a57.md b/gcc/config/arm/cortex-a57.md
> index 37912db464315a0d70835b81991e8e07a4d9db89..9b5970a0b647abc364b733cb4e2e22ae03056235 100644
> --- a/gcc/config/arm/cortex-a57.md
> +++ b/gcc/config/arm/cortex-a57.md
> @@ -726,7 +726,7 @@
>
> (define_insn_reservation "cortex_a57_fp_cpys" 4
> (and (eq_attr "tune" "cortexa57")
> - (eq_attr "type" "fmov"))
> + (eq_attr "type" "fmov,fcsel"))
> "(ca57_cx1|ca57_cx2)")
>
> (define_insn_reservation "cortex_a57_fp_divs" 12
>