This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][wwwdocs][AArch64] Add entry for target attributes and pragmas
- From: Kyrill Tkachov <kyrylo dot tkachov at arm dot com>
- To: Gerald Pfeifer <gerald at pfeifer dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Marcus Shawcroft <Marcus dot Shawcroft at arm dot com>, James Greenhalgh <James dot Greenhalgh at arm dot com>
- Date: Wed, 02 Sep 2015 09:24:57 +0100
- Subject: Re: [PATCH][wwwdocs][AArch64] Add entry for target attributes and pragmas
- Authentication-results: sourceware.org; auth=none
- References: <55E5B6BC dot 6020005 at arm dot com> <alpine dot LSU dot 2 dot 20 dot 1509011927150 dot 2597 at tuna dot site>
Hi Gerald,
On 01/09/15 18:29, Gerald Pfeifer wrote:
On Tue, 1 Sep 2015, Kyrill Tkachov wrote:
This wwwdocs patch adds an entry to the GCC 6 changes page about the
aarch64 target attributes and pragmas support.
Thanks for thinking of this, Kyrill.
Thanks for the feedback.
Index: htdocs/gcc-6/changes.html
===================================================================
+ <li>
+ The AArch64 port now supports target attributes and pragmas. Please
+ refer to the documentation for details of available attributes and
+ pragmas as well as usage instructions.
+ </li>
Here, isn't the second sentence the default assumption anyway,
that is, do we need to highlight it specifically or can we omit
it?
My thinking was that when we introduce some new command-line option we list it
here and give a short description of it (new -mcpu values, for example).
However, here we introduce about 10 new target attributes and pragmas and listing
them all would make this entry too long for my liking so as a shorthand for listing
them all I chose to point to the documentation.
Unless you feel strongly against this reasoning I'd like to commit the
patch as is within 48 hours.
Please consider this feedback (and "let's keep it as is" is a
fine response, too) and go ahead.
Gerald