[PATCH 1/2] aarch64: Remove AARCH64_FEATURE_F16 from AARCH64_ARCH_V8_2

Siddhesh Poyarekar siddhesh@gotplt.org
Fri Oct 27 12:49:00 GMT 2017


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



More information about the Binutils mailing list