This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
[PATCH 1/2] aarch64: Remove AARCH64_FEATURE_F16 from AARCH64_ARCH_V8_2
- From: Siddhesh Poyarekar <siddhesh at gotplt dot org>
- To: binutils at sourceware dot org
- Cc: Siddhesh Poyarekar <siddhesh dot poyarekar at linaro dot org>
- Date: Fri, 27 Oct 2017 18:18:38 +0530
- Subject: [PATCH 1/2] aarch64: Remove AARCH64_FEATURE_F16 from AARCH64_ARCH_V8_2
- Authentication-results: sourceware.org; auth=none
From: Siddhesh Poyarekar <siddhesh.poyarekar@linaro.org>
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/opcode/aarch64.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/include/opcode/aarch64.h b/include/opcode/aarch64.h
index c5788e2..ea45a41 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.7.5