[ARM] Clarify doc for .arch/.cpu

Message ID CAKnkMGs5cmD_6CU43QWwtvYfKhMrCGvrR2F7vwMJtg3-Q1L7gw@mail.gmail.com
State New
Headers show
Series
  • [ARM] Clarify doc for .arch/.cpu
Related show

Commit Message

Thomas Preudhomme July 25, 2018, 9:09 a.m.
Documentation for .arch and .cpu directives currently says that it
accepts the same name as -march/-mcpu command-line options respectively.
However it only accept the architecture/CPU part of those options: it
does not accept specifying an extension which is done via
.arch_extension. This patch clarifies that the extension is not
accepted.

ChangeLog entry is as follows:

*** gas/ChangeLog ***

2018-07-25  Thomas Preud'homme  <thomas.preudhomme@linaro.org>

    * doc/c-arm.texi (.arch directive): Clarify that name must not include
    an extension.
    (.cpu directive): Likewise.

Testing: Successfully built the info, PDF and HTML gas documentation.

Is this ok for master?

Best regards,

Thomas

Comments

Nick Clifton July 25, 2018, 10:01 a.m. | #1
Hi Thomas,

> 2018-07-25  Thomas Preud'homme  <thomas.preudhomme@linaro.org>

> 

>     * doc/c-arm.texi (.arch directive): Clarify that name must not include

>     an extension.

>     (.cpu directive): Likewise.


Approved - please apply.

Cheers
  Nick

Patch

From e50d39a7e433dfe79409a42355ad994cf1e25669 Mon Sep 17 00:00:00 2001
From: Thomas Preud'homme <thomas.preudhomme@arm.com>
Date: Wed, 25 Jul 2018 09:51:51 +0100
Subject: [PATCH] [PATCH, ARM] Clarify doc for .arch/.cpu

Documentation for .arch and .cpu directives currently says that it
accepts the same name as -march/-mcpu command-line options respectively.
However it only accept the architecture/CPU part of those options: it
does not accept specifying an extension which is done via
.arch_extension. This patch clarifies that the extension is not
accepted.

ChangeLog entry is as follows:

*** gas/ChangeLog ***

2018-07-25  Thomas Preud'homme  <thomas.preudhomme@linaro.org>

	* doc/c-arm.texi (.arch directive): Clarify that name must not include
	an extension.
	(.cpu directive): Likewise.

Testing: Successfully built the info, PDF and HTML gas documentation.

Is this ok for master?

Best regards,

Thomas
---
 gas/doc/c-arm.texi | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/gas/doc/c-arm.texi b/gas/doc/c-arm.texi
index d2f6fe1809..a316c267e2 100644
--- a/gas/doc/c-arm.texi
+++ b/gas/doc/c-arm.texi
@@ -624,7 +624,8 @@  boundary).  This is for compatibility with ARM's own assembler.
 @cindex @code{.arch} directive, ARM
 @item .arch @var{name}
 Select the target architecture.  Valid values for @var{name} are the same as
-for the @option{-march} command-line option.
+for the @option{-march} command-line option without the instruction set
+extension.
 
 Specifying @code{.arch} clears any previously selected architecture
 extensions.
@@ -663,7 +664,8 @@  selects Thumb, with the value 32 selecting ARM.
 @cindex @code{.cpu} directive, ARM
 @item .cpu @var{name}
 Select the target processor.  Valid values for @var{name} are the same as
-for the @option{-mcpu} command-line option.
+for the @option{-mcpu} command-line option without the instruction set
+extension.
 
 Specifying @code{.cpu} clears any previously selected architecture
 extensions.
-- 
2.18.0