-----Original Message-----
From: Anton Youdkevitch <anton.youdkevi...@bell-sw.com>
Sent: 27 April 2020 18:21
To: Kyrylo Tkachov <kyrylo.tkac...@arm.com>
Cc: gcc-patches@gcc.gnu.org; Richard Earnshaw
<richard.earns...@arm.com>; James Greenhalgh
<james.greenha...@arm.com>; Richard Sandiford
<richard.sandif...@arm.com>; jjo...@marvell.com
Subject: Re: [PATCH v5] aarch64: Add TX3 machine model
On Mon, Apr 27, 2020 at 04:34:49PM +0000, Kyrylo Tkachov wrote:
Hi Anton,
-----Original Message-----
From: Anton Youdkevitch <anton.youdkevi...@bell-sw.com>
Sent: 27 April 2020 11:24
To: gcc-patches@gcc.gnu.org
Cc: Richard Earnshaw <richard.earns...@arm.com>; Kyrylo Tkachov
<kyrylo.tkac...@arm.com>; James Greenhalgh
<james.greenha...@arm.com>; Richard Sandiford
<richard.sandif...@arm.com>; jjo...@marvell.com
Subject: [PATCH v5] aarch64: Add TX3 machine model
Here is the patch introducing thunderx3t110 machine model
for the scheduler. A name for the new chip was added to the
list of the names to be recognized as a valid parameter for
mcpu and mtune flags. Added the TX3 tuning table and cost
model tables.
Added the new chip name to the documentation. Fixed copyright
names and dates.
Lowering the chip capabilities to v8.3 to be on the safe side.
Bootstrapped on AArch64.
2020-04-27 Anton Youdkevitch <anton.youdkevi...@bell-sw.com>
* config/aarch64/aarch64-cores.def: Add the chip name.
* config/aarch64/aarch64-tune.md: Regenerated.
* config/aarch64/aarch64.c: Add tuning table for the chip.
* gcc/config/aarch64/aarch64-cost-tables.h: Add cost tables.
* config/aarch64/thunderx3t110.md: New file: add the new
machine model for the scheduler
* config/aarch64/aarch64.md: Include the new model.
* doc/invoke.texi: Add the new name to the list
---
gcc/config/aarch64/aarch64-cores.def | 3 +
gcc/config/aarch64/aarch64-cost-tables.h | 103 +++
gcc/config/aarch64/aarch64-tune.md | 2 +-
gcc/config/aarch64/aarch64.c | 83 ++
gcc/config/aarch64/aarch64.md | 1 +
gcc/config/aarch64/thunderx3t110.md | 686 +++++++++++++++
gcc/doc/invoke.texi | 2 +-
7 files changed, 878 insertions(+), 2 deletions(-)
diff --git a/gcc/config/aarch64/aarch64-cores.def
b/gcc/config/aarch64/aarch64-cores.def
index ea9b98b..4d8605a 100644
--- a/gcc/config/aarch64/aarch64-cores.def
+++ b/gcc/config/aarch64/aarch64-cores.def
@@ -95,6 +95,9 @@ AARCH64_CORE("vulcan", vulcan, thunderx2t99,
8_1A, AARCH64_FL_FOR_ARCH8_1 | AA
/* Cavium ('C') cores. */
AARCH64_CORE("thunderx2t99", thunderx2t99, thunderx2t99, 8_1A,
AARCH64_FL_FOR_ARCH8_1 | AARCH64_FL_CRYPTO, thunderx2t99, 0x43,
0x0af, -1)
+/* Marvell cores (TX3). */
+AARCH64_CORE("thunderx3t110", thunderx3t110, thunderx3t110, 8_3A,
AARCH64_FL_FOR_ARCH8_3 | AARCH64_FL_CRYPTO | AARCH64_FL_RCPC |
AARCH64_FL_SM4 | AARCH64_FL_SHA3 | AARCH64_FL_F16FML |
AARCH64_FL_RCPC8_4, thunderx3t110, 0x43, 0x0b8, 0x0a)
+
Please move this to a new section with a comment /* ARMv8.3-A
Architecture processors*/
So that we're consistent with the format of the file.
Fixed.