diff options
author | Tamar Christina <tamar.christina@arm.com> | 2025-01-16 19:25:26 +0000 |
---|---|---|
committer | Tamar Christina <tamar.christina@arm.com> | 2025-01-16 19:25:26 +0000 |
commit | 1ff85affe46623fe1a970de95887df22f4da9d16 (patch) | |
tree | 668f47284a241e5efddf653df4d677786cd20fda /gcc/config/aarch64 | |
parent | 773beeaafb0ea31bd4e308b64781731d64b571ce (diff) | |
download | gcc-1ff85affe46623fe1a970de95887df22f4da9d16.zip gcc-1ff85affe46623fe1a970de95887df22f4da9d16.tar.gz gcc-1ff85affe46623fe1a970de95887df22f4da9d16.tar.bz2 |
AArch64: have -mcpu=native detect architecture extensions for unknown non-homogenous systems [PR113257]
in g:e91a17fe39c39e98cebe6e1cbc8064ee6846a3a7 we added the ability for
-mcpu=native on unknown CPUs to still enable architecture extensions.
This has worked great but was only added for homogenous systems.
However the same thing works for big.LITTLE as in such system the cores must
have the same extensions otherwise it doesn't fundamentally work.
i.e. task migration from one core to the other wouldn't work.
This extends the same handling to non-homogenous systems.
gcc/ChangeLog:
PR target/113257
* config/aarch64/driver-aarch64.cc (get_cpu_from_id, DEFAULT_CPU): New.
(host_detect_local_cpu): Use it.
gcc/testsuite/ChangeLog:
PR target/113257
* gcc.target/aarch64/cpunative/info_34: New test.
* gcc.target/aarch64/cpunative/native_cpu_34.c: New test.
* gcc.target/aarch64/cpunative/info_35: New test.
* gcc.target/aarch64/cpunative/native_cpu_35.c: New test.
Co-authored-by: Richard Sandiford <richard.sandiford@arm.com>
Diffstat (limited to 'gcc/config/aarch64')
-rw-r--r-- | gcc/config/aarch64/driver-aarch64.cc | 52 |
1 files changed, 38 insertions, 14 deletions
diff --git a/gcc/config/aarch64/driver-aarch64.cc b/gcc/config/aarch64/driver-aarch64.cc index 45fce67..9d99554 100644 --- a/gcc/config/aarch64/driver-aarch64.cc +++ b/gcc/config/aarch64/driver-aarch64.cc @@ -60,6 +60,7 @@ struct aarch64_core_data #define ALL_VARIANTS ((unsigned)-1) /* Default architecture to use if -mcpu=native did not detect a known CPU. */ #define DEFAULT_ARCH "8A" +#define DEFAULT_CPU "generic-armv8-a" #define AARCH64_CORE(CORE_NAME, CORE_IDENT, SCHED, ARCH, FLAGS, COSTS, IMP, PART, VARIANT) \ { CORE_NAME, #ARCH, IMP, PART, VARIANT, feature_deps::cpu_##CORE_IDENT }, @@ -106,6 +107,19 @@ get_arch_from_id (const char* id) return NULL; } +/* Return an aarch64_core_data for the cpu described + by ID, or NULL if ID describes something we don't know about. */ + +static const aarch64_core_data * +get_cpu_from_id (const char* name) +{ + for (unsigned i = 0; aarch64_cpu_data[i].name != NULL; i++) + if (strcmp (name, aarch64_cpu_data[i].name) == 0) + return &aarch64_cpu_data[i]; + + return NULL; +} + /* Check wether the CORE array is the same as the big.LITTLE BL_CORE. For an example CORE={0xd08, 0xd03} and BL_CORE=AARCH64_BIG_LITTLE (0xd08, 0xd03) will return true. */ @@ -403,18 +417,11 @@ host_detect_local_cpu (int argc, const char **argv) || variants[0] == aarch64_cpu_data[i].variant)) break; - if (aarch64_cpu_data[i].name == NULL) + if (arch) { - auto arch_info = get_arch_from_id (DEFAULT_ARCH); - - gcc_assert (arch_info); - - res = concat ("-march=", arch_info->name, NULL); - default_flags = arch_info->flags; - } - else if (arch) - { - const char *arch_id = aarch64_cpu_data[i].arch; + const char *arch_id = (aarch64_cpu_data[i].name + ? aarch64_cpu_data[i].arch + : DEFAULT_ARCH); auto arch_info = get_arch_from_id (arch_id); /* We got some arch indentifier that's not in aarch64-arches.def? */ @@ -424,12 +431,15 @@ host_detect_local_cpu (int argc, const char **argv) res = concat ("-march=", arch_info->name, NULL); default_flags = arch_info->flags; } - else + else if (cpu || aarch64_cpu_data[i].name) { - default_flags = aarch64_cpu_data[i].flags; + auto cpu_info = (aarch64_cpu_data[i].name + ? &aarch64_cpu_data[i] + : get_cpu_from_id (DEFAULT_CPU)); + default_flags = cpu_info->flags; res = concat ("-m", cpu ? "cpu" : "tune", "=", - aarch64_cpu_data[i].name, + cpu_info->name, NULL); } } @@ -449,6 +459,20 @@ host_detect_local_cpu (int argc, const char **argv) break; } } + + /* On big.LITTLE if we find any unknown CPUs we can still pick arch + features as the cores should have the same features. So just pick + the feature flags from any of the cpus. */ + if (cpu && aarch64_cpu_data[i].name == NULL) + { + auto cpu_info = get_cpu_from_id (DEFAULT_CPU); + + gcc_assert (cpu_info); + + res = concat ("-mcpu=", cpu_info->name, NULL); + default_flags = cpu_info->flags; + } + if (!res) goto not_found; } |