public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* [PATCH v2][Binutils] aarch64: Enable Cortex-A720 CPU
@ 2023-08-15 15:58 Richard Ball
  2023-08-16 10:12 ` Richard Sandiford
  0 siblings, 1 reply; 2+ messages in thread
From: Richard Ball @ 2023-08-15 15:58 UTC (permalink / raw)
  To: binutils, Richard Sandiford, Richard Earnshaw, Marcus Shawcroft

[-- Attachment #1: Type: text/plain, Size: 407 bytes --]

v2: Add missing PROFILE feature flag.

This patch adds support for the Cortex-A720 CPU to binutils.

No regressions on aarch64-none-elf.

Ok for binutils-master?

bfd/ChangeLog:

	* cpu-aarch64.c: Add Cortex-A720.

gas/ChangeLog:

         * NEWS: Update docs.
         * config/tc-aarch64.c: Add Cortex-A720.
         * doc/c-aarch64.texi: Update docs.
	* testsuite/gas/aarch64/cpu-cortex-a720.d: New test.

[-- Attachment #2: Cortex-A720Binutils.patch --]
[-- Type: text/plain, Size: 2469 bytes --]

diff --git a/bfd/cpu-aarch64.c b/bfd/cpu-aarch64.c
index 75a0338b90cda6ec28c1aeedb5350166c58c2db0..df0d8c8c0dd0297501cc83abc1400b5ce37509e6 100644
--- a/bfd/cpu-aarch64.c
+++ b/bfd/cpu-aarch64.c
@@ -73,7 +73,8 @@ processors[] =
   { bfd_mach_aarch64,	  "cortex-a65"	    },
   { bfd_mach_aarch64,	  "cortex-a65ae"    },
   { bfd_mach_aarch64,	  "cortex-a76ae"    },
-  { bfd_mach_aarch64,	  "cortex-a77"	    }
+  { bfd_mach_aarch64,	  "cortex-a77"	    },
+  { bfd_mach_aarch64,	  "cortex-a720"	    }
 };
 
 static bool
diff --git a/gas/NEWS b/gas/NEWS
index 07ba75661052842554a478ee612b9c7a087cec1b..41eb1ab89feb8a6b375b8ff6e619701232489f00 100644
--- a/gas/NEWS
+++ b/gas/NEWS
@@ -12,6 +12,8 @@
 
 * Add support for Cortex-A520 for AArch64.
 
+* Add support for Cortex-A720 for AArch64.
+
 Changes in 2.41:
 
 * Add support for Intel FRED instructions.
diff --git a/gas/config/tc-aarch64.c b/gas/config/tc-aarch64.c
index e692161105a5c75fab92f707a114c20c9531f18f..19fbc7c46466ab30725e9ae16ae576aea4de5d6d 100644
--- a/gas/config/tc-aarch64.c
+++ b/gas/config/tc-aarch64.c
@@ -10237,6 +10237,11 @@ static const struct aarch64_cpu_option_table aarch64_cpus[] = {
                   | AARCH64_FEATURE_MEMTAG
                   | AARCH64_FEATURE_SVE2_BITPERM),
    "Cortex-A710"},
+  {"cortex-a720", AARCH64_FEATURE (AARCH64_ARCH_V9_2,
+                  AARCH64_FEATURE_MEMTAG
+		  | AARCH64_FEATURE_PROFILE
+                  | AARCH64_FEATURE_SVE2_BITPERM),
+   "Cortex-A720"},
   {"ares", AARCH64_FEATURE (AARCH64_ARCH_V8_2,
 				  AARCH64_FEATURE_RCPC | AARCH64_FEATURE_F16
 				  | AARCH64_FEATURE_DOTPROD
diff --git a/gas/doc/c-aarch64.texi b/gas/doc/c-aarch64.texi
index ed6b46e042290d806d98728d725b62520644e246..5e8ac0b4e7c6737e52edd6daadcd06e487572245 100644
--- a/gas/doc/c-aarch64.texi
+++ b/gas/doc/c-aarch64.texi
@@ -74,6 +74,7 @@ on the target processor.  The following processor names are recognized:
 @code{cortex-a510},
 @code{cortex-a520},
 @code{cortex-a710},
+@code{cortex-a720},
 @code{ares},
 @code{exynos-m1},
 @code{falkor},
diff --git a/gas/testsuite/gas/aarch64/cpu-cortex-a720.d b/gas/testsuite/gas/aarch64/cpu-cortex-a720.d
new file mode 100644
index 0000000000000000000000000000000000000000..c0b1f2b33750c4a63833f5f104bcd4dced93a9d2
--- /dev/null
+++ b/gas/testsuite/gas/aarch64/cpu-cortex-a720.d
@@ -0,0 +1,6 @@
+# name: Assemble and dump for cortex-a720 CPU
+# source: nop-asm.s
+# as: -mcpu=cortex-a720
+# objdump: -d -mcortex-a720
+
+#...

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH v2][Binutils] aarch64: Enable Cortex-A720 CPU
  2023-08-15 15:58 [PATCH v2][Binutils] aarch64: Enable Cortex-A720 CPU Richard Ball
@ 2023-08-16 10:12 ` Richard Sandiford
  0 siblings, 0 replies; 2+ messages in thread
From: Richard Sandiford @ 2023-08-16 10:12 UTC (permalink / raw)
  To: Richard Ball; +Cc: binutils, Richard Earnshaw, Marcus Shawcroft

Richard Ball <richard.ball@arm.com> writes:
> v2: Add missing PROFILE feature flag.
>
> This patch adds support for the Cortex-A720 CPU to binutils.
>
> No regressions on aarch64-none-elf.
>
> Ok for binutils-master?
>
> bfd/ChangeLog:
>
> 	* cpu-aarch64.c: Add Cortex-A720.
>
> gas/ChangeLog:
>
>          * NEWS: Update docs.
>          * config/tc-aarch64.c: Add Cortex-A720.
>          * doc/c-aarch64.texi: Update docs.
> 	* testsuite/gas/aarch64/cpu-cortex-a720.d: New test.

OK, thanks.  I was tempted to say:

>
> diff --git a/bfd/cpu-aarch64.c b/bfd/cpu-aarch64.c
> index 75a0338b90cda6ec28c1aeedb5350166c58c2db0..df0d8c8c0dd0297501cc83abc1400b5ce37509e6 100644
> --- a/bfd/cpu-aarch64.c
> +++ b/bfd/cpu-aarch64.c
> @@ -73,7 +73,8 @@ processors[] =
>    { bfd_mach_aarch64,	  "cortex-a65"	    },
>    { bfd_mach_aarch64,	  "cortex-a65ae"    },
>    { bfd_mach_aarch64,	  "cortex-a76ae"    },
> -  { bfd_mach_aarch64,	  "cortex-a77"	    }
> +  { bfd_mach_aarch64,	  "cortex-a77"	    },
> +  { bfd_mach_aarch64,	  "cortex-a720"	    }
>  };
>  
>  static bool
> diff --git a/gas/NEWS b/gas/NEWS
> index 07ba75661052842554a478ee612b9c7a087cec1b..41eb1ab89feb8a6b375b8ff6e619701232489f00 100644
> --- a/gas/NEWS
> +++ b/gas/NEWS
> @@ -12,6 +12,8 @@
>  
>  * Add support for Cortex-A520 for AArch64.
>  
> +* Add support for Cortex-A720 for AArch64.
> +

...that it might be better to combine these two entries, but I see
both styles are widely used.

Thanks,
Richard

>  Changes in 2.41:
>  
>  * Add support for Intel FRED instructions.
> diff --git a/gas/config/tc-aarch64.c b/gas/config/tc-aarch64.c
> index e692161105a5c75fab92f707a114c20c9531f18f..19fbc7c46466ab30725e9ae16ae576aea4de5d6d 100644
> --- a/gas/config/tc-aarch64.c
> +++ b/gas/config/tc-aarch64.c
> @@ -10237,6 +10237,11 @@ static const struct aarch64_cpu_option_table aarch64_cpus[] = {
>                    | AARCH64_FEATURE_MEMTAG
>                    | AARCH64_FEATURE_SVE2_BITPERM),
>     "Cortex-A710"},
> +  {"cortex-a720", AARCH64_FEATURE (AARCH64_ARCH_V9_2,
> +                  AARCH64_FEATURE_MEMTAG
> +		  | AARCH64_FEATURE_PROFILE
> +                  | AARCH64_FEATURE_SVE2_BITPERM),
> +   "Cortex-A720"},
>    {"ares", AARCH64_FEATURE (AARCH64_ARCH_V8_2,
>  				  AARCH64_FEATURE_RCPC | AARCH64_FEATURE_F16
>  				  | AARCH64_FEATURE_DOTPROD
> diff --git a/gas/doc/c-aarch64.texi b/gas/doc/c-aarch64.texi
> index ed6b46e042290d806d98728d725b62520644e246..5e8ac0b4e7c6737e52edd6daadcd06e487572245 100644
> --- a/gas/doc/c-aarch64.texi
> +++ b/gas/doc/c-aarch64.texi
> @@ -74,6 +74,7 @@ on the target processor.  The following processor names are recognized:
>  @code{cortex-a510},
>  @code{cortex-a520},
>  @code{cortex-a710},
> +@code{cortex-a720},
>  @code{ares},
>  @code{exynos-m1},
>  @code{falkor},
> diff --git a/gas/testsuite/gas/aarch64/cpu-cortex-a720.d b/gas/testsuite/gas/aarch64/cpu-cortex-a720.d
> new file mode 100644
> index 0000000000000000000000000000000000000000..c0b1f2b33750c4a63833f5f104bcd4dced93a9d2
> --- /dev/null
> +++ b/gas/testsuite/gas/aarch64/cpu-cortex-a720.d
> @@ -0,0 +1,6 @@
> +# name: Assemble and dump for cortex-a720 CPU
> +# source: nop-asm.s
> +# as: -mcpu=cortex-a720
> +# objdump: -d -mcortex-a720
> +
> +#...

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-08-16 10:12 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-08-15 15:58 [PATCH v2][Binutils] aarch64: Enable Cortex-A720 CPU Richard Ball
2023-08-16 10:12 ` Richard Sandiford

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).