> On 28 Jul 2025, at 10:39, Jennifer Schmitz <jschm...@nvidia.com> wrote:
>
> External email: Use caution opening links or attachments
>
>
>> 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.
Pushed to trunk: b6c9ed6756f82135740f1b363791000580beb38c
>>
>>>> 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.
Pushed to releases/gcc-15: 54cfe505872e61c24100ccd75bcbb056454cb2f0
> 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
