Skip to content

Commit 5252880

Browse files
ptomsichtstellar
authored andcommitted
[AArch64] Ampere1 does not support MTE
The initial support for the Ampere1 mistakenly signalled support for the MTE feature. However, the core does not include the optional MTE functionality. Update the target parser to not include MTE for Ampere1. Reviewed By: dmgreen Differential Revision: https://reviews.llvm.org/D125191 (cherry picked from commit 91b24b0)
1 parent 53433dd commit 5252880

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

llvm/include/llvm/Support/AArch64TargetParser.def

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -291,8 +291,7 @@ AARCH64_CPU_NAME("a64fx", ARMV8_2A, FK_CRYPTO_NEON_FP_ARMV8, false,
291291
AARCH64_CPU_NAME("carmel", ARMV8_2A, FK_CRYPTO_NEON_FP_ARMV8, false,
292292
AArch64::AEK_FP16)
293293
AARCH64_CPU_NAME("ampere1", ARMV8_6A, FK_CRYPTO_NEON_FP_ARMV8, false,
294-
(AArch64::AEK_FP16 | AArch64::AEK_MTE | AArch64::AEK_SB |
295-
AArch64::AEK_SSBS))
294+
(AArch64::AEK_FP16 | AArch64::AEK_SB | AArch64::AEK_SSBS))
296295
// Invalid CPU
297296
AARCH64_CPU_NAME("invalid", INVALID, FK_INVALID, true, AArch64::AEK_INVALID)
298297
#undef AARCH64_CPU_NAME

llvm/unittests/Support/TargetParserTest.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1192,7 +1192,7 @@ INSTANTIATE_TEST_SUITE_P(
11921192
AArch64::AEK_RDM | AArch64::AEK_RCPC | AArch64::AEK_DOTPROD |
11931193
AArch64::AEK_SM4 | AArch64::AEK_SHA3 | AArch64::AEK_BF16 |
11941194
AArch64::AEK_SHA2 | AArch64::AEK_AES | AArch64::AEK_I8MM |
1195-
AArch64::AEK_MTE | AArch64::AEK_SSBS | AArch64::AEK_SB,
1195+
AArch64::AEK_SSBS | AArch64::AEK_SB,
11961196
"8.6-A"),
11971197
ARMCPUTestParams(
11981198
"neoverse-512tvb", "armv8.4-a", "crypto-neon-fp-armv8",

0 commit comments

Comments
 (0)