public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH][GCC] aarch64: enable cortex-x2 CPU
@ 2021-09-22  8:38 Przemyslaw Wirkus
  2021-10-01 12:17 ` Kyrylo Tkachov
  0 siblings, 1 reply; 4+ messages in thread
From: Przemyslaw Wirkus @ 2021-09-22  8:38 UTC (permalink / raw)
  To: gcc-patches
  Cc: Richard Earnshaw, Richard Sandiford, Marcus Shawcroft, Kyrylo Tkachov

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

Patch is adding 'cortex-x2' to -mcpu command line option.

OK for master?

gcc/ChangeLog:

2021-09-02  Przemyslaw Wirkus  <przemyslaw.wirkus@arm.com>

	* config/aarch64/aarch64-cores.def (AARCH64_CORE): New
	Cortex-X2 core.
	* config/aarch64/aarch64-tune.md: Regenerate.
	* doc/invoke.texi: Update docs.

[-- Attachment #2: rb14825.patch --]
[-- Type: application/octet-stream, Size: 3409 bytes --]

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)
+
 #undef AARCH64_CORE
diff --git a/gcc/config/aarch64/aarch64-tune.md b/gcc/config/aarch64/aarch64-tune.md
index 1de5a178745bb1161c3018d40168a77b67855a34..47f9c2889f53f089b2200ee040a939d66a71f86b 100644
--- a/gcc/config/aarch64/aarch64-tune.md
+++ b/gcc/config/aarch64/aarch64-tune.md
@@ -1,5 +1,5 @@
 ;; -*- buffer-read-only: t -*-
 ;; Generated automatically by gentune.sh from aarch64-cores.def
 (define_attr "tune"
-	"cortexa34,cortexa35,cortexa53,cortexa57,cortexa72,cortexa73,thunderx,thunderxt88p1,thunderxt88,octeontx,octeontxt81,octeontxt83,thunderxt81,thunderxt83,emag,xgene1,falkor,qdf24xx,exynosm1,phecda,thunderx2t99p1,vulcan,thunderx2t99,cortexa55,cortexa75,cortexa76,cortexa76ae,cortexa77,cortexa78,cortexa78ae,cortexa78c,cortexa65,cortexa65ae,cortexx1,ares,neoversen1,neoversee1,octeontx2,octeontx2t98,octeontx2t96,octeontx2t93,octeontx2f95,octeontx2f95n,octeontx2f95mm,a64fx,tsv110,thunderx3t110,zeus,neoversev1,saphira,neoversen2,cortexa57cortexa53,cortexa72cortexa53,cortexa73cortexa35,cortexa73cortexa53,cortexa75cortexa55,cortexa76cortexa55,cortexr82,cortexa510,cortexa710"
+	"cortexa34,cortexa35,cortexa53,cortexa57,cortexa72,cortexa73,thunderx,thunderxt88p1,thunderxt88,octeontx,octeontxt81,octeontxt83,thunderxt81,thunderxt83,emag,xgene1,falkor,qdf24xx,exynosm1,phecda,thunderx2t99p1,vulcan,thunderx2t99,cortexa55,cortexa75,cortexa76,cortexa76ae,cortexa77,cortexa78,cortexa78ae,cortexa78c,cortexa65,cortexa65ae,cortexx1,ares,neoversen1,neoversee1,octeontx2,octeontx2t98,octeontx2t96,octeontx2t93,octeontx2f95,octeontx2f95n,octeontx2f95mm,a64fx,tsv110,thunderx3t110,zeus,neoversev1,saphira,neoversen2,cortexa57cortexa53,cortexa72cortexa53,cortexa73cortexa35,cortexa73cortexa53,cortexa75cortexa55,cortexa76cortexa55,cortexr82,cortexa510,cortexa710,cortexx2"
 	(const (symbol_ref "((enum attr_tune) aarch64_tune)")))
diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
index baa3c185bc6c9196473b9fcf75b672fe2f2e74a1..eb1993e298e1da6035fa94dc508c123107ac1135 100644
--- a/gcc/doc/invoke.texi
+++ b/gcc/doc/invoke.texi
@@ -18384,8 +18384,8 @@ performance of the code.  Permissible values for this option are:
 @samp{cortex-a57.cortex-a53}, @samp{cortex-a72.cortex-a53},
 @samp{cortex-a73.cortex-a35}, @samp{cortex-a73.cortex-a53},
 @samp{cortex-a75.cortex-a55}, @samp{cortex-a76.cortex-a55},
-@samp{cortex-r82}, @samp{cortex-x1}, @samp{cortex-a510},
-@samp{cortex-a710}, @samp{native}.
+@samp{cortex-r82}, @samp{cortex-x1}, @samp{cortex-x2},
+@samp{cortex-a510}, @samp{cortex-a710}, @samp{native}.
 
 The values @samp{cortex-a57.cortex-a53}, @samp{cortex-a72.cortex-a53},
 @samp{cortex-a73.cortex-a35}, @samp{cortex-a73.cortex-a53},

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

* RE: [PATCH][GCC] aarch64: enable cortex-x2 CPU
  2021-09-22  8:38 [PATCH][GCC] aarch64: enable cortex-x2 CPU Przemyslaw Wirkus
@ 2021-10-01 12:17 ` Kyrylo Tkachov
  2021-10-01 12:18   ` Kyrylo Tkachov
  0 siblings, 1 reply; 4+ messages in thread
From: Kyrylo Tkachov @ 2021-10-01 12:17 UTC (permalink / raw)
  To: Przemyslaw Wirkus, gcc-patches
  Cc: Richard Earnshaw, Richard Sandiford, Marcus Shawcroft



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

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

* RE: [PATCH][GCC] aarch64: enable cortex-x2 CPU
  2021-10-01 12:17 ` Kyrylo Tkachov
@ 2021-10-01 12:18   ` Kyrylo Tkachov
  2021-10-01 21:47     ` Przemyslaw Wirkus
  0 siblings, 1 reply; 4+ messages in thread
From: Kyrylo Tkachov @ 2021-10-01 12:18 UTC (permalink / raw)
  To: Przemyslaw Wirkus, gcc-patches
  Cc: Richard Earnshaw, Richard Sandiford, Marcus Shawcroft



> -----Original Message-----
> From: Kyrylo Tkachov
> Sent: Friday, October 1, 2021 1:17 PM
> To: Przemyslaw Wirkus <Przemyslaw.Wirkus@arm.com>; gcc-
> patches@gcc.gnu.org
> Cc: Richard Earnshaw <Richard.Earnshaw@arm.com>; Richard Sandiford
> <Richard.Sandiford@arm.com>; Marcus Shawcroft
> <Marcus.Shawcroft@arm.com>
> Subject: RE: [PATCH][GCC] aarch64: enable cortex-x2 CPU
> 
> 
> 
> > -----Original Message-----
> > From: Przemyslaw Wirkus <Przemyslaw.Wirkus@arm.com>
> > Sent: Wednesday, September 22, 2021 9:38 AM
> > To: gcc-patches@gcc.gnu.org
> > Cc: Richard Earnshaw <Richard.Earnshaw@arm.com>; Richard Sandiford
> > <Richard.Sandiford@arm.com>; Marcus Shawcroft
> > <Marcus.Shawcroft@arm.com>; Kyrylo Tkachov
> <Kyrylo.Tkachov@arm.com>
> > 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  <przemyslaw.wirkus@arm.com>
> >
> > 	* 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

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

* RE: [PATCH][GCC] aarch64: enable cortex-x2 CPU
  2021-10-01 12:18   ` Kyrylo Tkachov
@ 2021-10-01 21:47     ` Przemyslaw Wirkus
  0 siblings, 0 replies; 4+ messages in thread
From: Przemyslaw Wirkus @ 2021-10-01 21:47 UTC (permalink / raw)
  To: Kyrylo Tkachov, gcc-patches
  Cc: Richard Earnshaw, Richard Sandiford, Marcus Shawcroft

> 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 <Przemyslaw.Wirkus@arm.com>; gcc-
> > patches@gcc.gnu.org
> > Cc: Richard Earnshaw <Richard.Earnshaw@arm.com>; Richard Sandiford
> > <Richard.Sandiford@arm.com>; Marcus Shawcroft
> > <Marcus.Shawcroft@arm.com>
> > Subject: RE: [PATCH][GCC] aarch64: enable cortex-x2 CPU
> >
> >
> >
> > > -----Original Message-----
> > > From: Przemyslaw Wirkus <Przemyslaw.Wirkus@arm.com>
> > > Sent: Wednesday, September 22, 2021 9:38 AM
> > > To: gcc-patches@gcc.gnu.org
> > > Cc: Richard Earnshaw <Richard.Earnshaw@arm.com>; Richard Sandiford
> > > <Richard.Sandiford@arm.com>; Marcus Shawcroft
> > > <Marcus.Shawcroft@arm.com>; Kyrylo Tkachov
> > <Kyrylo.Tkachov@arm.com>
> > > 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  <przemyslaw.wirkus@arm.com>
> > >
> > > 	* 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

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

end of thread, other threads:[~2021-10-01 21:48 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-22  8:38 [PATCH][GCC] aarch64: enable cortex-x2 CPU Przemyslaw Wirkus
2021-10-01 12:17 ` Kyrylo Tkachov
2021-10-01 12:18   ` Kyrylo Tkachov
2021-10-01 21:47     ` Przemyslaw Wirkus

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).