diff mbox

[AArch64] Add missing fcsel in Cortex-A57 scheduler

Message ID DB3PR08MB008991A651ED2D80654D46FF83580@DB3PR08MB0089.eurprd08.prod.outlook.com
State New
Headers show

Commit Message

Wilco Dijkstra June 2, 2016, 4:09 p.m. UTC 
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.

---

Comments

James Greenhalgh June 2, 2016, 4:24 p.m. UTC | #1 
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
>
Kyrill Tkachov June 6, 2016, 4:21 p.m. UTC | #2 
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
>
diff mbox

Patch

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