This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH, gas/ARM, ping] Add support for Cortex-M23

From: Thomas Preudhomme <thomas dot preudhomme at foss dot arm dot com>
To: "Richard Earnshaw (lists)" <Richard dot Earnshaw at arm dot com>, Nick Clifton <nickc at redhat dot com>, Alan Modra <amodra at gmail dot com>, binutils at sourceware dot org
Date: Wed, 2 Nov 2016 10:11:07 +0000
Subject: Re: [PATCH, gas/ARM, ping] Add support for Cortex-M23
Authentication-results: sourceware.org; auth=none
References: <fad1cdc5-0960-9d3e-d121-fa12db14fc9f@foss.arm.com> <b1f4cdf1-95f1-ea8d-1487-6ed844a01d55@arm.com> <ce166e92-f8e4-3d06-5e4f-3f68235f1adb@foss.arm.com> <76418d58-1412-9ce4-09c0-05efdb21cf2b@arm.com>

Nick: ping?

Is this ok for ARM to mention support for new Cortex-M23 and Cortex-M33processors in gas/NEWS?


Best regards,

Thomas

On 31/10/16 13:45, Richard Earnshaw (lists) wrote:

On 28/10/16 15:32, Thomas Preudhomme wrote:

On 27/10/16 13:44, Richard Earnshaw (lists) wrote:

OK.  Don't forget to update the NEWS file.


I wasn't sure I should. I didn't see any entry for new processor support
in the NEWS file. It seems we (ARM but also the community in general I
think) only mention architecture.

I'm happy to change our behavior (and I think it makes sense to announce
any interface change) but I just want an extra confirmation from you:


OK, I'd missed that we don't normally mention CPU names.  I'll let Nick
make the call on which version he wants (NEWS or no-NEWS).

R.


*** gas/ChangeLog ***

2016-10-27  Thomas Preud'homme  <thomas.preudhomme@arm.com>

        * config/tc-arm.c (cortex-m23): Declare new processor.
        * doc/c-arm.texi (-mcpu ARM command line option): Document new
        Cortex-M23 processor.
        * NEWS: Mention Cortex-M23 support.


Ok for master?

Best regards,

Thomas

add_gas_cortex-m23_support.patch


diff --git a/gas/NEWS b/gas/NEWS
index 2dd2245d6e7fd762d02c7bbeff11aab7e536ecb5..a2a62563f867edf26462a08a23950b179710d288 100644
--- a/gas/NEWS
+++ b/gas/NEWS
@@ -1,5 +1,9 @@
 -*- text -*-

+Changes in 2.28:
+
+* Add support for the Cortex-M23 processor.
+
 Changes in 2.27:

 * Default to --enable-compressed-debug-sections=gas for Linux/x86 targets.
diff --git a/gas/config/tc-arm.c b/gas/config/tc-arm.c
index 73f8396396ff85276dc5bc92f79764d6d194b3d3..69dfe989133cbdc12f2da9d2864e6788eace3353 100644
--- a/gas/config/tc-arm.c
+++ b/gas/config/tc-arm.c
@@ -25406,6 +25406,8 @@ static const struct arm_cpu_option_table arm_cpus[] =
   ARM_CPU_OPT ("cortex-r8",	ARM_ARCH_V7R_IDIV,
 						 FPU_ARCH_VFP_V3D16,
 								  "Cortex-R8"),
+  ARM_CPU_OPT ("cortex-m23",	ARM_ARCH_V8M_BASE,
+						 FPU_NONE,	  "Cortex-M23"),
   ARM_CPU_OPT ("cortex-m7",	ARM_ARCH_V7EM,	 FPU_NONE,	  "Cortex-M7"),
   ARM_CPU_OPT ("cortex-m4",	ARM_ARCH_V7EM,	 FPU_NONE,	  "Cortex-M4"),
   ARM_CPU_OPT ("cortex-m3",	ARM_ARCH_V7M,	 FPU_NONE,	  "Cortex-M3"),
diff --git a/gas/doc/c-arm.texi b/gas/doc/c-arm.texi
index 0cf7390dc5f28dc34b33c377ff59f06f5aee2a89..565ef73250e957e2da11cc261ec9d4b5ccb9c7c2 100644
--- a/gas/doc/c-arm.texi
+++ b/gas/doc/c-arm.texi
@@ -131,6 +131,7 @@ recognized:
 @code{cortex-r5},
 @code{cortex-r7},
 @code{cortex-r8},
+@code{cortex-m23},
 @code{cortex-m7},
 @code{cortex-m4},
 @code{cortex-m3},

Follow-Ups:
- Re: [PATCH, gas/ARM, ping] Add support for Cortex-M23
  - From: Nick Clifton

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]