This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [ARM][gas] Add crypto-neon-fp-armv8.1 as an fpu option.
- From: Matthew Wahab <matthew dot wahab at foss dot arm dot com>
- To: binutils at sourceware dot org
- Date: Wed, 15 Jul 2015 15:32:42 +0100
- Subject: Re: [ARM][gas] Add crypto-neon-fp-armv8.1 as an fpu option.
- Authentication-results: sourceware.org; auth=none
- References: <5583E246 dot 7040505 at arm dot com> <5583E5C3 dot 5070104 at arm dot com> <5583EB82 dot 10300 at arm dot com>
Ping. Updated patch attached.
On 19/06/15 11:14, Matthew Wahab wrote:
On 19/06/15 10:49, Matthew Wahab wrote:
On 19/06/15 10:35, Matthew Wahab wrote:
Support for ARMv8.1 Adv.SIMD was recently added to gas as architecture
extensions and as the -mfpu option neon-fp-armv8.1. This patch adds the -mfpu
option crypto-neon-fp-armv8.1 as neon-fp-armv8.1 with the crypto extension
enabled.
Unfortunately I got the documentation wrong for this and the neon-fp-armv8.1 option.
The manual has the command line options ending with 'v8-1' rather than 'v8.1'.
I'll redo the patch to fix both.
Attached, the updated patch.
Tested for arm-none-linux-gnueabihf with check-binutils and check-gas. Checked the
HTML documentation.
Ok for trunk?
Matthew
gas/
2015-06-19 Matthew Wahab <matthew.wahab@arm.com>
* config/tc-arm.c (arm_fpus): Add crypto-neon-fp-armv8.1.
* doc/c-arm.texi (-mfpu=): Likewise. Correct the entry for
neon-fp-armv8.1.
diff --git a/gas/config/tc-arm.c b/gas/config/tc-arm.c
index 1793965..b1e51f5 100644
--- a/gas/config/tc-arm.c
+++ b/gas/config/tc-arm.c
@@ -24989,6 +24989,8 @@ static const struct arm_option_fpu_value_table arm_fpus[] =
{"crypto-neon-fp-armv8",
FPU_ARCH_CRYPTO_NEON_VFP_ARMV8},
{"neon-fp-armv8.1", FPU_ARCH_NEON_VFP_ARMV8_1},
+ {"crypto-neon-fp-armv8.1",
+ FPU_ARCH_CRYPTO_NEON_VFP_ARMV8_1},
{NULL, ARM_ARCH_NONE}
};
diff --git a/gas/doc/c-arm.texi b/gas/doc/c-arm.texi
index 0c3b9f9..7fadd28 100644
--- a/gas/doc/c-arm.texi
+++ b/gas/doc/c-arm.texi
@@ -274,9 +274,10 @@ The following format options are recognized:
@code{neon},
@code{neon-vfpv4},
@code{neon-fp-armv8},
-@code{crypto-neon-fp-armv8}.
+@code{crypto-neon-fp-armv8},
+@code{neon-fp-armv8.1}
and
-@code{neon-fp-armv8-1},
+@code{crypto-neon-fp-armv8.1}.
In addition to determining which instructions are assembled, this option
also affects the way in which the @code{.double} assembler directive behaves