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: Kyrill Tkachov <kyrylo dot tkachov at foss dot arm dot com>
- To: Wilco Dijkstra <Wilco dot Dijkstra at arm dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 06 Jun 2016 17:21:46 +0100
- Subject: Re: [PATCH][AArch64] Add missing fcsel in Cortex-A57 scheduler
- Authentication-results: sourceware.org; auth=none
- References: <DB3PR08MB008991A651ED2D80654D46FF83580 at DB3PR08MB0089 dot eurprd08 dot prod dot outlook dot com>
On 02/06/16 17:09, Wilco Dijkstra wrote:
The Cortex-A57 scheduler is missing fcsel, so add it.
OK for commit?
ChangeLog:
2016-06-02 Wilco Dijkstra <wdijkstr@arm.com>
* config/arm/cortex-a57.md (cortex_a57_fp_cpys): Add fcsel.
Ok from an arm perspective too.
Thanks,
Kyrill
---
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