diff mbox series

[Arm] Set Armv7-A tune to Cortex-A53

Message ID VI1PR0801MB212768C16115750910A67E71834D0@VI1PR0801MB2127.eurprd08.prod.outlook.com
State New
Headers show
Series [Arm] Set Armv7-A tune to Cortex-A53 | expand

Commit Message

Wilco Dijkstra Nov. 18, 2019, 3:55 p.m. UTC 
By default Armv7-A tunes for Cortex-A8. This is an ancient core
today and the settings are no longer useful for newer cores. So
switch to Cortex-A53 tuning since it works well across a wide range
of modern cores.

On SPECINT2006 the performance gain is 0.7% compared to Cortex-A8 tuning,
and codesize reduces by 0.2%.

OK for commit?

ChangeLog:

2019-11-15  Wilco Dijkstra  <wdijkstr@arm.com>

        * config/arm/arm-cpus.in (armv7): Set tune to Cortex-A53.
        (armv7-a): Likewise.
        (armv7ve): Likewise.
---

Comments

Richard Earnshaw (lists) Nov. 18, 2019, 11:24 p.m. UTC | #1 
On 18/11/2019 15:55, Wilco Dijkstra wrote:
> By default Armv7-A tunes for Cortex-A8. This is an ancient core
> today and the settings are no longer useful for newer cores. So
> switch to Cortex-A53 tuning since it works well across a wide range
> of modern cores.
> 
> On SPECINT2006 the performance gain is 0.7% compared to Cortex-A8 tuning,
> and codesize reduces by 0.2%.
> 
> OK for commit?
> 
> ChangeLog:
> 
> 2019-11-15  Wilco Dijkstra  <wdijkstr@arm.com>
> 
>         * config/arm/arm-cpus.in (armv7): Set tune to Cortex-A53.
>         (armv7-a): Likewise.
>         (armv7ve): Likewise.

Seems reasonable.  OK.

R.

> ---
> 
> diff --git a/gcc/config/arm/arm-cpus.in b/gcc/config/arm/arm-cpus.in
> index 50379a0a10a96c7fd501b401adbd679737b7a322..44e6cc6bdb604eb992d28dd74845edfac9abaad5 100644
> --- a/gcc/config/arm/arm-cpus.in
> +++ b/gcc/config/arm/arm-cpus.in
> @@ -415,7 +415,7 @@ begin arch armv6s-m
>  end arch armv6s-m
>  
>  begin arch armv7
> - tune for cortex-a8
> + tune for cortex-a53
>   tune flags CO_PROC
>   base 7
>   isa ARMv7
> @@ -426,7 +426,7 @@ begin arch armv7
>  end arch armv7
>  
>  begin arch armv7-a
> - tune for cortex-a8
> + tune for cortex-a53
>   tune flags CO_PROC
>   base 7A
>   profile A
> @@ -451,7 +451,7 @@ begin arch armv7-a
>  end arch armv7-a
>  
>  begin arch armv7ve
> - tune for cortex-a8
> + tune for cortex-a53
>   tune flags CO_PROC
>   base 7A
>   profile A
>
diff mbox series

Patch

diff --git a/gcc/config/arm/arm-cpus.in b/gcc/config/arm/arm-cpus.in
index 50379a0a10a96c7fd501b401adbd679737b7a322..44e6cc6bdb604eb992d28dd74845edfac9abaad5 100644
--- a/gcc/config/arm/arm-cpus.in
+++ b/gcc/config/arm/arm-cpus.in
@@ -415,7 +415,7 @@  begin arch armv6s-m
 end arch armv6s-m
 
 begin arch armv7
- tune for cortex-a8
+ tune for cortex-a53
  tune flags CO_PROC
  base 7
  isa ARMv7
@@ -426,7 +426,7 @@  begin arch armv7
 end arch armv7
 
 begin arch armv7-a
- tune for cortex-a8
+ tune for cortex-a53
  tune flags CO_PROC
  base 7A
  profile A
@@ -451,7 +451,7 @@  begin arch armv7-a
 end arch armv7-a
 
 begin arch armv7ve
- tune for cortex-a8
+ tune for cortex-a53
  tune flags CO_PROC
  base 7A
  profile A