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

2021-10-01 Thread Przemyslaw Wirkus via Gcc-patches 
> Subject: RE: [PATCH][GCC] aarch64: enable cortex-x2 CPU
> 
> 
> 
> > -Original Message-
> > From: Kyrylo Tkachov
> > Sent: Friday, October 1, 2021 1:17 PM
> > To: Przemyslaw Wirkus ; gcc-
> > patc...@gcc.gnu.org
> > Cc: Richard Earnshaw ; Richard Sandiford
> > ; Marcus Shawcroft
> > 
> > Subject: RE: [PATCH][GCC] aarch64: enable cortex-x2 CPU
> >
> >
> >
> > > -Original Message-
> > > From: Przemyslaw Wirkus 
> > > Sent: Wednesday, September 22, 2021 9:38 AM
> > > To: gcc-patches@gcc.gnu.org
> > > Cc: Richard Earnshaw ; Richard Sandiford
> > > ; Marcus Shawcroft
> > > ; Kyrylo Tkachov
> > 
> > > Subject: [PATCH][GCC] aarch64: enable cortex-x2 CPU
> > >
> > > Patch is adding 'cortex-x2' to -mcpu command line option.
> > >
> > > OK for master?
> > >
> > > gcc/ChangeLog:
> > >
> > > 2021-09-02  Przemyslaw Wirkus  
> > >
> > >   * config/aarch64/aarch64-cores.def (AARCH64_CORE): New
> > >   Cortex-X2 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
> >
> a8027e92fa8f7554e2b19d00f7c85c6ed48a92e5..34d9646ab6a32a19e7cd09d9
> > 5594b59278d02920 100644
> > --- a/gcc/config/aarch64/aarch64-cores.def
> > +++ b/gcc/config/aarch64/aarch64-cores.def
> > @@ -168,4 +168,6 @@ AARCH64_CORE("cortex-a510",  cortexa510,
> > cortexa55, 9A,  AARCH64_FL_FOR_ARCH9 |
> >
> >  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)
> >
> > +AARCH64_CORE("cortex-x2",  cortexx2, cortexa55, 9A,
> > AARCH64_FL_FOR_ARCH9 | AARCH64_FL_SVE2_BITPERM |
> AARCH64_FL_MEMTAG |
> > AARCH64_FL_I8MM | AARCH64_FL_BF16, neoversen2, 0x41, 0xd48, -1)
> > +
> >
> > Let's use cortexa57 for scheduling here for now.
> 
> I should have said, ok with that change.

commit 257d2890a769a8aa564d079170377e637e07acb1

> Kyrill
> > Thanks,
> > Kyrill

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

2021-10-01 Thread Kyrylo Tkachov via Gcc-patches 



> -Original Message-
> From: Kyrylo Tkachov
> Sent: Friday, October 1, 2021 1:17 PM
> To: Przemyslaw Wirkus ; gcc-
> patc...@gcc.gnu.org
> Cc: Richard Earnshaw ; Richard Sandiford
> ; Marcus Shawcroft
> 
> Subject: RE: [PATCH][GCC] aarch64: enable cortex-x2 CPU
> 
> 
> 
> > -Original Message-
> > From: Przemyslaw Wirkus 
> > Sent: Wednesday, September 22, 2021 9:38 AM
> > To: gcc-patches@gcc.gnu.org
> > Cc: Richard Earnshaw ; Richard Sandiford
> > ; Marcus Shawcroft
> > ; Kyrylo Tkachov
> 
> > Subject: [PATCH][GCC] aarch64: enable cortex-x2 CPU
> >
> > Patch is adding 'cortex-x2' to -mcpu command line option.
> >
> > OK for master?
> >
> > gcc/ChangeLog:
> >
> > 2021-09-02  Przemyslaw Wirkus  
> >
> > * config/aarch64/aarch64-cores.def (AARCH64_CORE): New
> > Cortex-X2 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
> a8027e92fa8f7554e2b19d00f7c85c6ed48a92e5..34d9646ab6a32a19e7cd09d9
> 5594b59278d02920 100644
> --- a/gcc/config/aarch64/aarch64-cores.def
> +++ b/gcc/config/aarch64/aarch64-cores.def
> @@ -168,4 +168,6 @@ AARCH64_CORE("cortex-a510",  cortexa510,
> cortexa55, 9A,  AARCH64_FL_FOR_ARCH9 |
> 
>  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)
> 
> +AARCH64_CORE("cortex-x2",  cortexx2, cortexa55, 9A,
> AARCH64_FL_FOR_ARCH9 | AARCH64_FL_SVE2_BITPERM |
> AARCH64_FL_MEMTAG | AARCH64_FL_I8MM | AARCH64_FL_BF16,
> neoversen2, 0x41, 0xd48, -1)
> +
> 
> Let's use cortexa57 for scheduling here for now.

I should have said, ok with that change.
Kyrill
> Thanks,
> Kyrill

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

2021-10-01 Thread Kyrylo Tkachov via Gcc-patches 



> -Original Message-
> From: Przemyslaw Wirkus 
> Sent: Wednesday, September 22, 2021 9:38 AM
> To: gcc-patches@gcc.gnu.org
> Cc: Richard Earnshaw ; Richard Sandiford
> ; Marcus Shawcroft
> ; Kyrylo Tkachov 
> Subject: [PATCH][GCC] aarch64: enable cortex-x2 CPU
> 
> Patch is adding 'cortex-x2' to -mcpu command line option.
> 
> OK for master?
> 
> gcc/ChangeLog:
> 
> 2021-09-02  Przemyslaw Wirkus  
> 
>   * config/aarch64/aarch64-cores.def (AARCH64_CORE): New
>   Cortex-X2 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 
a8027e92fa8f7554e2b19d00f7c85c6ed48a92e5..34d9646ab6a32a19e7cd09d95594b59278d02920
 100644
--- a/gcc/config/aarch64/aarch64-cores.def
+++ b/gcc/config/aarch64/aarch64-cores.def
@@ -168,4 +168,6 @@ AARCH64_CORE("cortex-a510",  cortexa510, cortexa55, 9A,  
AARCH64_FL_FOR_ARCH9 |
 
 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)
 
+AARCH64_CORE("cortex-x2",  cortexx2, cortexa55, 9A,  AARCH64_FL_FOR_ARCH9 | 
AARCH64_FL_SVE2_BITPERM | AARCH64_FL_MEMTAG | AARCH64_FL_I8MM | 
AARCH64_FL_BF16, neoversen2, 0x41, 0xd48, -1)
+

Let's use cortexa57 for scheduling here for now.
Thanks,
Kyrill