Hi Przemek,
> -----Original Message-----
> From: Przemyslaw Wirkus <przemyslaw.wir...@arm.com>
> Sent: Wednesday, September 22, 2021 9:35 AM
> To: gcc-patches@gcc.gnu.org
> Cc: Richard Earnshaw <richard.earns...@arm.com>; Richard Sandiford
> <richard.sandif...@arm.com>; Marcus Shawcroft
> <marcus.shawcr...@arm.com>; Kyrylo Tkachov <kyrylo.tkac...@arm.com>
> Subject: [PATCH][GCC] aarch64: enable cortex-a510 CPU
>
> Patch is adding 'cortex-a510' to -mcpu command line option.
>
> gcc/ChangeLog:
>
> 2021-09-02 Przemyslaw Wirkus <przemyslaw.wir...@arm.com>
>
> * config/aarch64/aarch64-cores.def (AARCH64_CORE): New
> Cortex-A510 core.
> * config/aarch64/aarch64-tune.md: Regenerate.
> * doc/invoke.texi: Update docs.
+/* Arm9.0-A Architecture Processors. */
Typo, should be "Armv9.0-a".
+
+/* Arm ('A') cores. */
+AARCH64_CORE("cortex-a510", cortexa510, cortexa55, 9A, AARCH64_FL_FOR_ARCH9
| AARCH64_FL_SVE2_BITPERM | AARCH64_FL_MEMTAG | AARCH64_FL_I8MM |
AARCH64_FL_BF16, neoversen2, 0x41, 0xd46, -1)
+
We'll need to update the tuning anyway once we do it properly, but for now I
think for the COSTS field (4th to last) we should go with cortexa53 rather than
neoversen2.
Ok with those changes.
Thanks,
Kyrill
