This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][doc][13/14] Document AArch64 target attributes and pragmas
- From: Sandra Loosemore <sandra at codesourcery dot com>
- To: Kyrill Tkachov <kyrylo dot tkachov at arm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Marcus Shawcroft <Marcus dot Shawcroft at arm dot com>, Richard Earnshaw <Richard dot Earnshaw at arm dot com>, James Greenhalgh <James dot Greenhalgh at arm dot com>
- Date: Fri, 17 Jul 2015 13:45:24 -0600
- Subject: Re: [PATCH][doc][13/14] Document AArch64 target attributes and pragmas
- Authentication-results: sourceware.org; auth=none
- References: <55A7CBEE dot 9060706 at arm dot com> <55A874CE dot 80205 at codesourcery dot com> <55A8F702 dot 2010602 at arm dot com>
On 07/17/2015 06:37 AM, Kyrill Tkachov wrote:
Hi Sandra,
On 17/07/15 04:21, Sandra Loosemore wrote:
On 07/16/2015 09:21 AM, Kyrill Tkachov wrote:
Hi all,
This patch adds the documentation for the AArch64 target attributes and
pragmas.
Ok for trunk?
The content looks OK, but I have a bunch of nit-picky comments about
grammar, typos, markup, etc....
Thanks for the detailed feedback!
Here's an updated version.
There are still a few things I previously noted, but that you missed in
preparing this version.
+the same behavior as that of the command line option
s/command line option/command-line option/g
https://gcc.gnu.org/codingconventions.html#Spelling
+where @var{@var{attr-string}} is one of the attribute strings specified above.
@var{@var{...}} should be @code{@var{...}}, I think.
+Note that CPU tuning options and attributes such as the @option{-mcpu=},
+@option{-mtune=} do not inhibit inlining unless the CPU specified by the
+@option{-mcpu=} option or the @option{cpu=} attribute conflicts with the
s/@option{cpu=} attribute/@code{cpu=} attribute/
+@smallexample
+#pragma GCC target("<string>")
s/<string>/@var{string}/
+@end smallexample
+
+where @code{@var{string}} can be any string accepted as an AArch64 target
+attribute. @xref{AArch64 Function Attributes}, for more details
+on the permissible values of @code{<string>}.
Here too.
OK with those things fixed.
-Sandra