RE: [PATCH][GCC] aarch64: enable cortex-a710 CPU

[Przemyslaw Wirkus via Gcc-patches]

> -----Original Message-----
> From: Kyrylo Tkachov <kyrylo.tkac...@arm.com>
> Sent: 01 October 2021 13:16
> To: Przemyslaw Wirkus <przemyslaw.wir...@arm.com>; gcc-
> patc...@gcc.gnu.org
> Cc: Richard Earnshaw <richard.earns...@arm.com>; Richard Sandiford
> <richard.sandif...@arm.com>; Marcus Shawcroft
> <marcus.shawcr...@arm.com>
> Subject: RE: [PATCH][GCC] aarch64: enable cortex-a710 CPU
> 
> 
> 
> > -----Original Message-----
> > From: Przemyslaw Wirkus <przemyslaw.wir...@arm.com>
> > Sent: Wednesday, September 22, 2021 9:37 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-a710 CPU
> >
> > Patch is adding 'cortex-a710' 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-A710 core.
> >     * config/aarch64/aarch64-tune.md: Regenerate.
> >     * doc/invoke.texi: Update docs.
> 
> diff --git a/gcc/config/aarch64/aarch64-cores.def
> b/gcc/config/aarch64/aarch64-cores.def
> index
> 478f7e1c8145365f42f43ad94d90c633aae66ebd..a8027e92fa8f7554e2b19d00f
> 7c85c6ed48a92e5 100644
> --- a/gcc/config/aarch64/aarch64-cores.def
> +++ b/gcc/config/aarch64/aarch64-cores.def
> @@ -166,4 +166,6 @@ AARCH64_CORE("cortex-r82", cortexr82, cortexa53,
> 8R, AARCH64_FL_FOR_ARCH8_R, cor
>  /* 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)
> 
> +AARCH64_CORE("cortex-a710",  cortexa710, cortexa55, 9A,
> +AARCH64_FL_FOR_ARCH9 | AARCH64_FL_SVE2_BITPERM |
> AARCH64_FL_MEMTAG |
> +AARCH64_FL_I8MM | AARCH64_FL_BF16, neoversen2, 0x41, 0xd47, -1)
> +
> 
> Again, we'd need to revisit big-core scheduling properly at some point, but
> for now I think for the scheduling field (3rd) we should use cortexa57 rather
> than cortexa55.
> Ok with that change.

commit f3cb2114d8b892fed0b6a717dab0a71d3da604bc

> Thanks,
> Kyrill