This is the mail archive of the mailing list for the binutils project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [ARM][gas] Add crypto-neon-fp-armv8.1 as an fpu option.

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

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?

2015-06-19  Matthew Wahab  <>

	* config/tc-arm.c (arm_fpus): Add crypto-neon-fp-armv8.1.
	* doc/c-arm.texi (-mfpu=): Likewise.  Correct the entry for

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[] =
   {"neon-fp-armv8.1",	FPU_ARCH_NEON_VFP_ARMV8_1},
+  {"crypto-neon-fp-armv8.1",
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:
 In addition to determining which instructions are assembled, this option
 also affects the way in which the @code{.double} assembler directive behaves

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]