diff mbox series

[1/5,Arm] Modify default tuning of armv8.1-m.main to use Cortex-M55

Message ID AM0PR08MB5121A47394DE2922ACCE9E6B924A0@AM0PR08MB5121.eurprd08.prod.outlook.com
State New
Headers show
Series Add support for conditional instructions (CSEL, CSINC etc.) for Armv8.1-M Mainline | expand

Commit Message

Omar Tahir Aug. 4, 2020, 4:09 p.m. UTC
Previously, compiling with -march=armv8.1-m.main would tune for Cortex-M7.
However, the Cortex-M7 only supports up to Armv7e-M. The Cortex-M55 is the
earliest CPU that supports Armv8.1-M Mainline so is more appropriate. This
also has the effect of changing the branch cost function used, which will be
necessary to correctly prioritise conditional instructions over branches in
the rest of this patch series.

Regression tested on arm-none-eabi.


gcc/ChangeLog:

2020-07-30: Omar Tahir <omar.tahir@arm.com>

                * config/arm/arm-cpus.in (armv8.1-m.main): Tune for Cortex-M55.

Comments

Kyrylo Tkachov Aug. 4, 2020, 4:38 p.m. UTC | #1
Hi Omar,

Ok, thanks.
I've pushed this to master.
Kyrill

From: Omar Tahir <Omar.Tahir@arm.com>
Sent: 04 August 2020 17:10
To: Kyrylo Tkachov <Kyrylo.Tkachov@arm.com>; nickc@redhat.com; Ramana Radhakrishnan <Ramana.Radhakrishnan@arm.com>; Richard Earnshaw <Richard.Earnshaw@arm.com>; gcc-patches@gcc.gnu.org
Subject: [PATCH 1/5][Arm] Modify default tuning of armv8.1-m.main to use Cortex-M55

Previously, compiling with -march=armv8.1-m.main would tune for Cortex-M7.
However, the Cortex-M7 only supports up to Armv7e-M. The Cortex-M55 is the
earliest CPU that supports Armv8.1-M Mainline so is more appropriate. This
also has the effect of changing the branch cost function used, which will be
necessary to correctly prioritise conditional instructions over branches in
the rest of this patch series.

Regression tested on arm-none-eabi.


gcc/ChangeLog:

2020-07-30: Omar Tahir <omar.tahir@arm.com<mailto:omar.tahir@arm.com>>

            * config/arm/arm-cpus.in (armv8.1-m.main): Tune for Cortex-M55.


diff --git a/gcc/config/arm/arm-cpus.in b/gcc/config/arm/arm-cpus.in
index 728be500b80..c98f8ede8fd 100644
--- a/gcc/config/arm/arm-cpus.in
+++ b/gcc/config/arm/arm-cpus.in
@@ -716,7 +716,7 @@ begin arch armv8-r
end arch armv8-r

begin arch armv8.1-m.main
- tune for cortex-m7
+ tune for cortex-m55
  tune flags CO_PROC
  base 8M_MAIN
  profile M
diff mbox series

Patch

diff --git a/gcc/config/arm/arm-cpus.in b/gcc/config/arm/arm-cpus.in
index 728be500b80..c98f8ede8fd 100644
--- a/gcc/config/arm/arm-cpus.in
+++ b/gcc/config/arm/arm-cpus.in
@@ -716,7 +716,7 @@  begin arch armv8-r
end arch armv8-r
 begin arch armv8.1-m.main
- tune for cortex-m7
+ tune for cortex-m55
  tune flags CO_PROC
  base 8M_MAIN
  profile M