I just joined the gcc-patches list, so I hope the mail software can parse this 
out with an "In-Reply-To" header.

I work for Marvell, and Anton's work is approved for submittal. I wrote the 
first version of the .md file. I'm certain we have a copyright assignment.in 
place, as we've had employees in the past six months submit changes, for 
example Steve Ellcey.

Joel Jones

>Hi Anton,
>
>> -----Original Message-----
>> From: Gcc-patches <gcc-patches-boun...@gcc.gnu.org> On Behalf Of Anton
>> Youdkevitch
>> Sent: 20 April 2020 19:29
>> To: gcc-patches@gcc.gnu.org
>> Cc: jo...@marvell.com
>> Subject: [PATCH v2] aarch64: Add TX3 machine model
>>
>> Here is the patch introducing thunderxt311 maching 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. The TX2 cost model was reused for TX3.
>>
>> The previously used "cryptic" name for the command line
>> parameter is replaced with the same "thunderxt311" name.
>>
>> Bootstrapped on AArch64.
>
>Thanks for the patch. I had meant to ask, do you have a copyright assignment 
>in place?
>We'd need one to accept a contribution of this size.
>Thanks,
>Kyrill
>
>>
>> 2020-04-20 Anton Youdkevitch <anton.youdkevi...@bell-sw.com>
>>
>>         * config/aarch64/aarch64-cores.def: Add the chip name.
>>         * config/aarch64/aarch64-tune.md: Regenerated.
>>         * gcc/config/aarch64/aarch64.c: Add the cost tables for the chip.
>>         * gcc/config/aarch64/thunderx3t11.md: New file: add the new
>>         machine model for the scheduler
>>         * gcc/config/aarch64/aarch64.md: Include the new model.
>>
>> ---
>>  gcc/config/aarch64/aarch64-cores.def |   3 +
>>  gcc/config/aarch64/aarch64-tune.md   |   2 +-
>>  gcc/config/aarch64/aarch64.c         |  27 +
>>  gcc/config/aarch64/aarch64.md        |   1 +
>>  gcc/config/aarch64/thunderx3t11.md   | 686 +++++++++++++++
>>  5 files changed, 718 insertions(+), 1 deletion(-)

Reply via email to