From c05e0c5af35ebbb1ca39bb1efffae2adecdfa253 Mon Sep 17 00:00:00 2001 From: Siddhesh Poyarekar Date: Thu, 2 Nov 2017 22:49:32 +0530 Subject: [PATCH] aarch64: Remove AARCH64_FEATURE_F16 from AARCH64_ARCH_V8_2 The FP16 feature is optional in ARMv8.2, so it is wrong to add it to the default AARCH64_ARCH_V8_2 feature flags. This patch makes the behaviour consistent with that of gcc, which also does not assume FP16 for ARMv8.2. include/ * opcode/aarch64.h (AARCH64_ARCH_V8_2): Drop AARCH64_FEATURE_F16. --- include/ChangeLog | 5 +++++ include/opcode/aarch64.h | 1 - 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/include/ChangeLog b/include/ChangeLog index fd2001f2d57..638ec43d97c 100644 --- a/include/ChangeLog +++ b/include/ChangeLog @@ -1,3 +1,8 @@ +2017-11-02 Siddhesh Poyarekar + + * include/opcode/aarch64.h (AARCH64_ARCH_V8_2): Drop + AARCH64_FEATURE_F16. + 2017-11-01 James Bowman * elf/ft32.h: Add R_FT32_RELAX, SC0, SC1, DIFF32. diff --git a/include/opcode/aarch64.h b/include/opcode/aarch64.h index c5788e2d09d..ea45a416e73 100644 --- a/include/opcode/aarch64.h +++ b/include/opcode/aarch64.h @@ -70,7 +70,6 @@ typedef uint32_t aarch64_insn; | AARCH64_FEATURE_RDMA) #define AARCH64_ARCH_V8_2 AARCH64_FEATURE (AARCH64_ARCH_V8_1, \ AARCH64_FEATURE_V8_2 \ - | AARCH64_FEATURE_F16 \ | AARCH64_FEATURE_RAS) #define AARCH64_ARCH_V8_3 AARCH64_FEATURE (AARCH64_ARCH_V8_2, \ AARCH64_FEATURE_V8_3 \ -- 2.30.2