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

2021-10-01 Thread Przemyslaw Wirkus via Gcc-patches 
> Hi Przemek,
> 
> > -Original Message-
> > From: Przemyslaw Wirkus 
> > Sent: Wednesday, September 22, 2021 9:35 AM
> > To: gcc-patches@gcc.gnu.org
> > Cc: Richard Earnshaw ; Richard Sandiford
> > ; Marcus Shawcroft
> > ; Kyrylo Tkachov
> 
> > 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  
> >
> > * 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.

commit 8aa3ab5a47664023d83ea5097a53a66bd6cbb978

> Thanks,
> Kyrill

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

2021-10-01 Thread Kyrylo Tkachov via Gcc-patches 
Hi Przemek,

> -Original Message-
> From: Przemyslaw Wirkus 
> Sent: Wednesday, September 22, 2021 9:35 AM
> To: gcc-patches@gcc.gnu.org
> Cc: Richard Earnshaw ; Richard Sandiford
> ; Marcus Shawcroft
> ; Kyrylo Tkachov 
> 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  
> 
>   * 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