[PATCH][wwwdocs][AArch64] Add entry for target attributes and pragmas

Kyrill Tkachov kyrylo.tkachov@arm.com
Tue Sep 15 12:45:00 GMT 2015


On 15/09/15 13:18, Christian Bruel wrote:
> Hi Kyril,
>
> On 09/02/2015 10:24 AM, Kyrill Tkachov wrote:
>> 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
>>>
> Hi Kyril,
>
> While you are at it, can you add it for the ARM entry a well ?

Sure, I'll send a patch

Kyrill

>
> many thanks
>
> Christian
>
>



More information about the Gcc-patches mailing list