[binutils-gdb] [arm] Document missing -mfpu entries.

Richard Earnshaw rearnsha@sourceware.org
Mon Mar 20 14:57:00 GMT 2017


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=d5e0ba9cdb0d002b97bab722cce673ce86cf7b11

commit d5e0ba9cdb0d002b97bab722cce673ce86cf7b11
Author: Richard Earnshaw <Richard.Earnshaw@arm.com>
Date:   Mon Mar 20 14:56:22 2017 +0000

    [arm] Document missing -mfpu entries.
    
    Nick pointed out that I hadn't documented the new -mfpu option
    neon-vfpv3 and mentioned that some others were missing.
    
    Having looked through the list only one (neon-fp16) really should be
    documented; the other two entries in the real table should not be
    documented as they are aliases kept for legacy compatibility reasons.
    This patch adds the missing entries and notes in the main table that
    the other two entries should not be documented.
    
    I've also fixed a small spelling error in the accompanying text.
    
    	* config/tc-arm.c (arm_fpus): Note entires that should not be
    	documented.
    	* doc/c-arm.texi (-mfpu): Add missing FPU entries for neon-vfpv3 and
    	neon-fp16.  Fix spelling error.

Diff:
---
 gas/ChangeLog       | 7 +++++++
 gas/config/tc-arm.c | 6 +++---
 gas/doc/c-arm.texi  | 4 +++-
 3 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/gas/ChangeLog b/gas/ChangeLog
index 6d7c403..6541eab 100644
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,5 +1,12 @@
 2017-03-20  Richard Earnshaw  <rearnsha@arm.com>
 
+	* config/tc-arm.c (arm_fpus): Note entires that should not be
+	documented.
+	* doc/c-arm.texi (-mfpu): Add missing FPU entries for neon-vfpv3 and
+	neon-fp16.  Fix spelling error.
+
+2017-03-20  Richard Earnshaw  <rearnsha@arm.com>
+
 	* config/tc-arm.c (arm_fpus): Add neon-vfpv3 as an alias for neon.
 
 2017-03-16  Rinat Zelig  <rinat@mellanox.com>
diff --git a/gas/config/tc-arm.c b/gas/config/tc-arm.c
index bb2985c..ef6b609 100644
--- a/gas/config/tc-arm.c
+++ b/gas/config/tc-arm.c
@@ -25781,7 +25781,7 @@ static const struct arm_option_fpu_value_table arm_fpus[] =
   {"softvfp+vfp",	FPU_ARCH_VFP_V2},
   {"vfp",		FPU_ARCH_VFP_V2},
   {"vfp9",		FPU_ARCH_VFP_V2},
-  {"vfp3",              FPU_ARCH_VFP_V3}, /* For backwards compatibility.  */
+  {"vfp3",		FPU_ARCH_VFP_V3}, /* Undocumented, use vfpv3.  */
   {"vfp10",		FPU_ARCH_VFP_V2},
   {"vfp10-r0",		FPU_ARCH_VFP_V1},
   {"vfpxd",		FPU_ARCH_VFP_V1xD},
@@ -25794,10 +25794,10 @@ static const struct arm_option_fpu_value_table arm_fpus[] =
   {"vfpv3xd-fp16",	FPU_ARCH_VFP_V3xD_FP16},
   {"arm1020t",		FPU_ARCH_VFP_V1},
   {"arm1020e",		FPU_ARCH_VFP_V2},
-  {"arm1136jfs",	FPU_ARCH_VFP_V2},
+  {"arm1136jfs",	FPU_ARCH_VFP_V2}, /* Undocumented, use arm1136jf-s.  */
   {"arm1136jf-s",	FPU_ARCH_VFP_V2},
   {"maverick",		FPU_ARCH_MAVERICK},
-  {"neon",              FPU_ARCH_VFP_V3_PLUS_NEON_V1},
+  {"neon",		FPU_ARCH_VFP_V3_PLUS_NEON_V1},
   {"neon-vfpv3",	FPU_ARCH_VFP_V3_PLUS_NEON_V1},
   {"neon-fp16",		FPU_ARCH_NEON_FP16},
   {"vfpv4",		FPU_ARCH_VFP_V4},
diff --git a/gas/doc/c-arm.texi b/gas/doc/c-arm.texi
index 29630a5..0a1392a 100644
--- a/gas/doc/c-arm.texi
+++ b/gas/doc/c-arm.texi
@@ -285,6 +285,8 @@ The following format options are recognized:
 @code{arm1136jf-s},
 @code{maverick},
 @code{neon},
+@code{neon-vfpv3},
+@code{neon-fp16},
 @code{neon-vfpv4},
 @code{neon-fp-armv8},
 @code{crypto-neon-fp-armv8},
@@ -297,7 +299,7 @@ also affects the way in which the @code{.double} assembler directive behaves
 when assembling little-endian code.
 
 The default is dependent on the processor selected.  For Architecture 5 or
-later, the default is to assembler for VFP instructions; for earlier
+later, the default is to assemble for VFP instructions; for earlier
 architectures the default is to assemble for FPA instructions.
 
 @cindex @code{-mthumb} command line option, ARM



More information about the Binutils-cvs mailing list