> On 28 Jul 2025, at 10:23, Kyrylo Tkachov <ktkac...@nvidia.com> wrote:
> 
> 
> 
>> On 27 Jul 2025, at 03:31, Andrew Pinski <pins...@gmail.com> wrote:
>> 
>> On Fri, Jul 25, 2025 at 5:14 AM Jennifer Schmitz <jschm...@nvidia.com> wrote:
>>> 
>>> This patch adds a new tuning model for the NVIDIA Olympus core.
>>> The values used here are based on the Software Optimization Guide
>>> that will be published imminently.
>>> 
>>> Bootstrapped and tested on aarch64-linux-gnu, no regression.
>>> 
>>> OK for trunk?
>> 
>> Ok from the trunk.
>> 
> 
> Ok from me as well.
> 
>>> OK to backport to GCC 15?
>> 
>> I think this should wait until after the release of GCC 15.2.0. RC1
>> comes out at the end of this coming week.
> 
> We’ve allowed backports of such non-default CPU heuristics in the past given 
> the long lead times in aligning GCC releases in distros with CPU releases.
> So as long as it doesn’t break the build or tests (Jennifer, it’s been tested 
> on the GCC 15 branch as well, right? I believe it has, but best to state 
> explicitly) I’m okay with pushing for GCC 15.2 as well, but give others a day 
> to reply.
Yes, I bootstrapped and tested on the GCC 15 branch.
Thanks,
Jennifer
> 
> Thanks,
> Kyrill
> 
>> 
>> Thanks,
>> Andrew
>> 
>>> 
>>> Signed-off-by: Jennifer Schmitz <jschm...@nvidia.com>
>>> Co-Authored-By: Dhruv Chawla <dhr...@nvidia.com>
>>> 
>>> gcc/ChangeLog:
>>> 
>>>       * config/aarch64/aarch64-cores.def (olympus): Use olympus tuning
>>>       model.
>>>       * config/aarch64/aarch64.cc: Include olympus.h.
>>>       * config/aarch64/tuning_models/olympus.h: New file.
>>> ---
>>> gcc/config/aarch64/aarch64-cores.def       |   2 +-
>>> gcc/config/aarch64/aarch64.cc              |   1 +
>>> gcc/config/aarch64/tuning_models/olympus.h | 210 +++++++++++++++++++++
>>> 3 files changed, 212 insertions(+), 1 deletion(-)
>>> create mode 100644 gcc/config/aarch64/tuning_models/olympus.h
>>> 
>>> diff --git a/gcc/config/aarch64/aarch64-cores.def 
>>> b/gcc/config/aarch64/aarch64-cores.def
>>> index 8040409d283..6f11cc0983a 100644
>>> --- a/gcc/config/aarch64/aarch64-cores.def
>>> +++ b/gcc/config/aarch64/aarch64-cores.def
>>> @@ -224,7 +224,7 @@ AARCH64_CORE("neoverse-v3ae", neoversev3ae, cortexa57, 
>>> V9_2A, (SVE2_BITPERM, RNG
>>> AARCH64_CORE("demeter", demeter, cortexa57, V9A, (I8MM, BF16, SVE2_BITPERM, 
>>> RNG, MEMTAG, PROFILE), neoversev2, 0x41, 0xd4f, -1)
>>> 
>>> /* NVIDIA ('N') cores. */
>>> -AARCH64_CORE("olympus", olympus, cortexa57, V9_2A, (SVE2_BITPERM, RNG, 
>>> LS64, MEMTAG, PROFILE, FAMINMAX, FP8FMA, FP8DOT2, FP8DOT4, LUT, SVE2_AES, 
>>> SVE2_SHA3, SVE2_SM4), neoversev3, 0x4e, 0x10, -1)
>>> +AARCH64_CORE("olympus", olympus, cortexa57, V9_2A, (SVE2_BITPERM, RNG, 
>>> LS64, MEMTAG, PROFILE, FAMINMAX, FP8FMA, FP8DOT2, FP8DOT4, LUT, SVE2_AES, 
>>> SVE2_SHA3, SVE2_SM4), olympus, 0x4e, 0x10, -1)
>>> 
>>> /* Armv9-A big.LITTLE processors.  */
>>> AARCH64_CORE("gb10",  gb10, cortexa57, V9_2A,  (SVE2_BITPERM, SVE2_AES, 
>>> SVE2_SHA3, SVE2_SM4, MEMTAG, PROFILE), cortexx925, 0x41, AARCH64_BIG_LITTLE 
>>> (0xd85, 0xd87), -1)
>>> diff --git a/gcc/config/aarch64/aarch64.cc b/gcc/config/aarch64/aarch64.cc
>>> index 0485f695941..58fef7198ba 100644
>>> --- a/gcc/config/aarch64/aarch64.cc
>>> +++ b/gcc/config/aarch64/aarch64.cc
>>> @@ -429,6 +429,7 @@ static const struct aarch64_flag_desc 
>>> aarch64_tuning_flags[] =
>>> #include "tuning_models/neoversev2.h"
>>> #include "tuning_models/neoversev3.h"
>>> #include "tuning_models/neoversev3ae.h"
>>> +#include "tuning_models/olympus.h"
>>> #include "tuning_models/a64fx.h"
>>> #include "tuning_models/fujitsu_monaka.h"
>>> 
>>> diff --git a/gcc/config/aarch64/tuning_models/olympus.h 
>>> b/gcc/config/aarch64/tuning_models/olympus.h
>>> new file mode 100644
>>> index 00000000000..268789db019
>>> --- /dev/null
>>> +++ b/gcc/config/aarch64/tuning_models/olympus.h
>>> @@ -0,0 +1,210 @@
>>> +/* Tuning model description for the NVIDIA Olympus core.
>>> +   Copyright The GNU Toolchain Authors.
>>> +
>>> +   This file is part of GCC.
>>> +
>>> +   GCC is free software; you can redistribute it and/or modify it
>>> +   under the terms of the GNU General Public License as published by
>>> +   the Free Software Foundation; either version 3, or (at your option)
>>> +   any later version.
>>> +
>>> +   GCC is distributed in the hope that it will be useful, but
>>> +   WITHOUT ANY WARRANTY; without even the implied warranty of
>>> +   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
>>> +   General Public License for more details.
>>> +
>>> +   You should have received a copy of the GNU General Public License
>>> +   along with GCC; see the file COPYING3.  If not see
>>> +   <http://www.gnu.org/licenses/>.  */
>>> +
>>> +#ifndef GCC_AARCH64_H_OLYMPUS
>>> +#define GCC_AARCH64_H_OLYMPUS
>>> +
>>> +#include "generic.h"
>>> +
>>> +static struct cpu_regmove_cost olympus_regmove_cost =
>>> +{
>>> +  1, /* GP2GP  */
>>> +  /* Spilling to int<->fp instead of memory is recommended so set
>>> +     realistic costs compared to memmov_cost.  */
>>> +  3, /* GP2FP  */
>>> +  3, /* FP2GP  */
>>> +  2 /* FP2FP  */
>>> +};
>>> +
>>> +static advsimd_vec_cost olympus_advsimd_vector_cost =
>>> +{
>>> +  2, /* int_stmt_cost  */
>>> +  2, /* fp_stmt_cost  */
>>> +  2, /* ld2_st2_permute_cost */
>>> +  2, /* ld3_st3_permute_cost  */
>>> +  3, /* ld4_st4_permute_cost  */
>>> +  2, /* permute_cost  */
>>> +  5, /* reduc_i8_cost  */
>>> +  3, /* reduc_i16_cost  */
>>> +  3, /* reduc_i32_cost  */
>>> +  2, /* reduc_i64_cost  */
>>> +  4, /* reduc_f16_cost  */
>>> +  4, /* reduc_f32_cost  */
>>> +  4, /* reduc_f64_cost  */
>>> +  2, /* store_elt_extra_cost  */
>>> +  8, /* vec_to_scalar_cost  */
>>> +  4, /* scalar_to_vec_cost  */
>>> +  6, /* align_load_cost  */
>>> +  6, /* unalign_load_cost  */
>>> +  1, /* unalign_store_cost  */
>>> +  1  /* store_cost  */
>>> +};
>>> +
>>> +static sve_vec_cost olympus_sve_vector_cost =
>>> +{
>>> +  {
>>> +    2, /* int_stmt_cost  */
>>> +    2, /* fp_stmt_cost  */
>>> +    2, /* ld2_st2_permute_cost  */
>>> +    3, /* ld3_st3_permute_cost  */
>>> +    3, /* ld4_st4_permute_cost  */
>>> +    2, /* permute_cost  */
>>> +    9, /* reduc_i8_cost  */
>>> +    8, /* reduc_i16_cost  */
>>> +    6, /* reduc_i32_cost  */
>>> +    2, /* reduc_i64_cost  */
>>> +    8, /* reduc_f16_cost  */
>>> +    6, /* reduc_f32_cost  */
>>> +    4, /* reduc_f64_cost  */
>>> +    2, /* store_elt_extra_cost  */
>>> +    8, /* vec_to_scalar_cost  */
>>> +    4, /* scalar_to_vec_cost  */
>>> +    4, /* align_load_cost  */
>>> +    6, /* unalign_load_cost  */
>>> +    1, /* unalign_store_cost  */
>>> +    1  /* store_cost  */
>>> +  },
>>> +  3, /* clast_cost  */
>>> +  10, /* fadda_f16_cost  */
>>> +  6, /* fadda_f32_cost  */
>>> +  4, /* fadda_f64_cost  */
>>> +  14, /* gather_load_x32_cost  */
>>> +  12, /* gather_load_x64_cost  */
>>> +  42, /* gather_load_x32_init_cost  */
>>> +  24, /* gather_load_x64_init_cost  */
>>> +  1 /* scatter_store_elt_cost  */
>>> +};
>>> +
>>> +static aarch64_scalar_vec_issue_info olympus_scalar_issue_info =
>>> +{
>>> +  4, /* loads_stores_per_cycle  */
>>> +  2, /* stores_per_cycle  */
>>> +  8, /* general_ops_per_cycle  */
>>> +  0, /* fp_simd_load_general_ops  */
>>> +  1 /* fp_simd_store_general_ops  */
>>> +};
>>> +
>>> +static aarch64_advsimd_vec_issue_info olympus_advsimd_issue_info =
>>> +{
>>> +  {
>>> +    3, /* loads_stores_per_cycle  */
>>> +    2, /* stores_per_cycle  */
>>> +    6, /* general_ops_per_cycle  */
>>> +    0, /* fp_simd_load_general_ops  */
>>> +    1 /* fp_simd_store_general_ops  */
>>> +  },
>>> +  2, /* ld2_st2_general_ops  */
>>> +  2, /* ld3_st3_general_ops  */
>>> +  3 /* ld4_st4_general_ops  */
>>> +};
>>> +
>>> +static aarch64_sve_vec_issue_info olympus_sve_issue_info =
>>> +{
>>> +  {
>>> +    {
>>> +      3, /* loads_stores_per_cycle  */
>>> +      2, /* stores_per_cycle  */
>>> +      6, /* general_ops_per_cycle  */
>>> +      0, /* fp_simd_load_general_ops  */
>>> +      1 /* fp_simd_store_general_ops  */
>>> +    },
>>> +    2, /* ld2_st2_general_ops  */
>>> +    2, /* ld3_st3_general_ops  */
>>> +    3 /* ld4_st4_general_ops  */
>>> +  },
>>> +  2, /* pred_ops_per_cycle  */
>>> +  1, /* while_pred_ops  */
>>> +  0, /* int_cmp_pred_ops  */
>>> +  0, /* fp_cmp_pred_ops  */
>>> +  1, /* gather_scatter_pair_general_ops  */
>>> +  1 /* gather_scatter_pair_pred_ops  */
>>> +};
>>> +
>>> +static aarch64_vec_issue_info olympus_vec_issue_info =
>>> +{
>>> +  &olympus_scalar_issue_info,
>>> +  &olympus_advsimd_issue_info,
>>> +  &olympus_sve_issue_info
>>> +};
>>> +
>>> +/* Olympus costs for vector insn classes.  */
>>> +static struct cpu_vector_cost olympus_vector_cost =
>>> +{
>>> +  1, /* scalar_int_stmt_cost  */
>>> +  2, /* scalar_fp_stmt_cost  */
>>> +  4, /* scalar_load_cost  */
>>> +  1, /* scalar_store_cost  */
>>> +  1, /* cond_taken_branch_cost  */
>>> +  1, /* cond_not_taken_branch_cost  */
>>> +  &olympus_advsimd_vector_cost, /* advsimd  */
>>> +  &olympus_sve_vector_cost, /* sve  */
>>> +  &olympus_vec_issue_info /* issue_info  */
>>> +};
>>> +
>>> +/* Olympus prefetch settings (which disable prefetch).  */
>>> +static cpu_prefetch_tune olympus_prefetch_tune =
>>> +{
>>> +  0,                   /* num_slots  */
>>> +  -1,                  /* l1_cache_size  */
>>> +  64,                  /* l1_cache_line_size  */
>>> +  -1,                  /* l2_cache_size  */
>>> +  true,                        /* prefetch_dynamic_strides */
>>> +  -1,                  /* minimum_stride */
>>> +  -1                   /* default_opt_level  */
>>> +};
>>> +
>>> +static struct tune_params olympus_tunings =
>>> +{
>>> +  &cortexa76_extra_costs,
>>> +  &generic_armv9_a_addrcost_table,
>>> +  &olympus_regmove_cost,
>>> +  &olympus_vector_cost,
>>> +  &generic_branch_cost,
>>> +  &generic_approx_modes,
>>> +  SVE_128, /* sve_width  */
>>> +  { 4, /* load_int.  */
>>> +    1, /* store_int.  */
>>> +    6, /* load_fp.  */
>>> +    3, /* store_fp.  */
>>> +    5, /* load_pred.  */
>>> +    1 /* store_pred.  */
>>> +  }, /* memmov_cost.  */
>>> +  10, /* issue_rate  */
>>> +  AARCH64_FUSE_NEOVERSE_BASE, /* fusible_ops  */
>>> +  "32:16",     /* function_align.  */
>>> +  "4",         /* jump_align.  */
>>> +  "32:16",     /* loop_align.  */
>>> +  8,   /* int_reassoc_width.  */
>>> +  6,   /* fp_reassoc_width.  */
>>> +  4,   /* fma_reassoc_width.  */
>>> +  6,   /* vec_reassoc_width.  */
>>> +  2,   /* min_div_recip_mul_sf.  */
>>> +  2,   /* min_div_recip_mul_df.  */
>>> +  0,   /* max_case_values.  */
>>> +  tune_params::AUTOPREFETCHER_WEAK,    /* autoprefetcher_model.  */
>>> +  (AARCH64_EXTRA_TUNE_BASE
>>> +   | AARCH64_EXTRA_TUNE_CSE_SVE_VL_CONSTANTS
>>> +   | AARCH64_EXTRA_TUNE_MATCHED_VECTOR_THROUGHPUT
>>> +   | AARCH64_EXTRA_TUNE_AVOID_PRED_RMW),       /* tune_flags.  */
>>> +  &olympus_prefetch_tune,
>>> +  AARCH64_LDP_STP_POLICY_ALWAYS,   /* ldp_policy_model.  */
>>> +  AARCH64_LDP_STP_POLICY_ALWAYS           /* stp_policy_model.  */
>>> +};
>>> +
>>> +#endif /* GCC_AARCH64_H_OLYMPUS.  */
>>> --
>>> 2.34.1


Reply via email to