Re: [PATCH] aarch64: Add TX3 machine model

2020-04-08 Thread Andrew Pinski via Gcc-patches
On Wed, Apr 8, 2020 at 11:06 AM Anton Youdkevitch
 wrote:
>
> 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.

It might make sense to use thunderx3 instead of thunderx3t11 now.
The main reason why I had used thunderx2t99 was because there was
going to be 2 Thunderx2 with two different costs but the second one
turned into OcteonTX2 instead.

Thanks,
Andrew

>
> Bootstrapped on AArch64.
>
> 2020-04-08  Anton Youdkevitch 
>
> * 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(-)
>


Re: [PATCH] aarch64: Add TX3 machine model

2020-04-09 Thread Anton Youdkevitch

Andrew,

On 09.4.2020 00:50 , Andrew Pinski wrote:

On Wed, Apr 8, 2020 at 11:06 AM Anton Youdkevitch
 wrote:

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.

It might make sense to use thunderx3 instead of thunderx3t11 now.
The main reason why I had used thunderx2t99 was because there was
going to be 2 Thunderx2 with two different costs but the second one
turned into OcteonTX2 instead.

We will revisited this. Thanks for the suggestion!

--
  Anton