This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
[ARM] Enable CRC by default for ARMv8.1 and later.
- From: Matthew Wahab <matthew dot wahab at foss dot arm dot com>
- To: "binutils at sourceware dot org" <binutils at sourceware dot org>
- Date: Mon, 7 Dec 2015 15:40:18 +0000
- Subject: [ARM] Enable CRC by default for ARMv8.1 and later.
- Authentication-results: sourceware.org; auth=none
Hello,
ARMv8.1 includes CRC as a required extension but the +crc feature isn't
enabled by -march=armv8.1-a as it should be. This patch fixes that.
Tested for arm-none-linux-gnueabihf with cross-compiled check-binutils
and check-gas.
Ok for trunk?
Matthew
opcode/include
2015-12-07 Matthew Wahab <matthew.wahab@arm.com>
* arm.h (ARM_ARCH_V8_1A): Add the CRC_EXT_ARMV8 co-processor
feature macro.
(ARM_ARCH_V8_2A): Likewise.
>From af850f93ea04a677425e3da7a2f180abef34829e Mon Sep 17 00:00:00 2001
From: Matthew Wahab <matthew.wahab@arm.com>
Date: Mon, 7 Dec 2015 12:05:49 +0000
Subject: [PATCH] [ARM] Enable CRC for ARMv8.1 and later.
Change-Id: I03584780b5878e0724ef778cc63258a840099869
---
include/opcode/arm.h | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/include/opcode/arm.h b/include/opcode/arm.h
index 5e7d683..0814d80 100644
--- a/include/opcode/arm.h
+++ b/include/opcode/arm.h
@@ -251,9 +251,11 @@
#define ARM_ARCH_V7M ARM_FEATURE_CORE_LOW (ARM_AEXT_V7M)
#define ARM_ARCH_V7EM ARM_FEATURE_CORE_LOW (ARM_AEXT_V7EM)
#define ARM_ARCH_V8A ARM_FEATURE_CORE_LOW (ARM_AEXT_V8A)
-#define ARM_ARCH_V8_1A ARM_FEATURE_CORE (ARM_AEXT_V8A, ARM_EXT2_PAN)
-#define ARM_ARCH_V8_2A ARM_FEATURE_CORE (ARM_AEXT_V8A, \
- ARM_EXT2_PAN | ARM_EXT2_V8_2A)
+#define ARM_ARCH_V8_1A ARM_FEATURE (ARM_AEXT_V8A, ARM_EXT2_PAN, \
+ CRC_EXT_ARMV8)
+#define ARM_ARCH_V8_2A ARM_FEATURE (ARM_AEXT_V8A, \
+ ARM_EXT2_PAN | ARM_EXT2_V8_2A, \
+ CRC_EXT_ARMV8)
/* Some useful combinations: */
#define ARM_ARCH_NONE ARM_FEATURE_LOW (0, 0)
--
2.1.4