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.

Ok with that change.
Kyrill

 /* ARMv8.2-A Architecture Processors.  */

Reply via email to