This is the mail archive of the
binutils-cvs@sourceware.org
mailing list for the binutils project.
[binutils-gdb] aarch64: Remove AARCH64_FEATURE_F16 from AARCH64_ARCH_V8_2
- From: Siddhesh Poyarekar <siddhesh at sourceware dot org>
- To: bfd-cvs at sourceware dot org
- Date: 2 Nov 2017 17:21:18 -0000
- Subject: [binutils-gdb] aarch64: Remove AARCH64_FEATURE_F16 from AARCH64_ARCH_V8_2
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=c05e0c5af35ebbb1ca39bb1efffae2adecdfa253
commit c05e0c5af35ebbb1ca39bb1efffae2adecdfa253
Author: Siddhesh Poyarekar <siddhesh@sourceware.org>
Date: Thu Nov 2 22:49:32 2017 +0530
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.
Diff:
---
include/ChangeLog | 5 +++++
include/opcode/aarch64.h | 1 -
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/include/ChangeLog b/include/ChangeLog
index fd2001f..638ec43 100644
--- a/include/ChangeLog
+++ b/include/ChangeLog
@@ -1,3 +1,8 @@
+2017-11-02 Siddhesh Poyarekar <siddhesh@sourceware.org>
+
+ * include/opcode/aarch64.h (AARCH64_ARCH_V8_2): Drop
+ AARCH64_FEATURE_F16.
+
2017-11-01 James Bowman <james.bowman@ftdichip.com>
* elf/ft32.h: Add R_FT32_RELAX, SC0, SC1, DIFF32.
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 \