From d5e0ba9cdb0d002b97bab722cce673ce86cf7b11 Mon Sep 17 00:00:00 2001 From: Richard Earnshaw Date: Mon, 20 Mar 2017 14:56:22 +0000 Subject: [PATCH] [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. --- 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 6d7c403b5a..6541eab189 100644 --- a/gas/ChangeLog +++ b/gas/ChangeLog @@ -1,3 +1,10 @@ +2017-03-20 Richard Earnshaw + + * 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 * config/tc-arm.c (arm_fpus): Add neon-vfpv3 as an alias for neon. diff --git a/gas/config/tc-arm.c b/gas/config/tc-arm.c index bb2985c782..ef6b609d08 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 29630a5671..0a1392a97d 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