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

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

[Kyrill Tkachov]

[-- Attachment #1: Type: text/plain, Size: 173 bytes --]

Hi all,

This wwwdocs patch adds an entry to the GCC 6 changes page about the aarch64 target attributes and pragmas support.
Ok for the wwwdocs repo?

Thanks,
Kyrill

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: wwwdocs-aarch64-target-attrs.patch --]
[-- Type: text/x-patch; name=wwwdocs-aarch64-target-attrs.patch, Size: 924 bytes --]

Index: htdocs/gcc-6/changes.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-6/changes.html,v
retrieving revision 1.21
diff -U 3 -r1.21 changes.html
--- htdocs/gcc-6/changes.html	13 Aug 2015 12:32:16 -0000	1.21
+++ htdocs/gcc-6/changes.html	19 Aug 2015 16:08:57 -0000
@@ -138,6 +138,15 @@
        the global offset table (GOT) is limited to 28KiB under the LP64 SysV ABI
        , and 15KiB under the ILP32 SysV ABI.
      </li>
+     <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>
+     <li>
+       Link-time optimization across translation units with different
+       target-specific options is now supported.
+     </li>
    </ul>
 <!-- <h3 id="arm">ARM</h3> -->
 

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

[Gerald Pfeifer]

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.

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?

Please consider this feedback (and "let's keep it as is" is a
fine response, too) and go ahead.

Gerald

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

[Kyrill Tkachov]

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
>

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

[Gerald Pfeifer]

On Wed, 2 Sep 2015, Kyrill Tkachov wrote:
> 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.

I can follow your reasoning, and anyway the 48 hours are way over ;-),
just have you considered adding a reference to the documentation (as a 
hyperlink to the respective section, if there is a good one, such as
https://gcc.gnu.org/onlinedocs/gcc/ARM-Pragmas.html#ARM-Pragmas )?

Gerald
> 

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

[Kyrill Tkachov]

Hi Gerald,

On 07/09/15 12:31, Gerald Pfeifer wrote:
> On Wed, 2 Sep 2015, Kyrill Tkachov wrote:
>> 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.
> I can follow your reasoning, and anyway the 48 hours are way over ;-),
> just have you considered adding a reference to the documentation (as a
> hyperlink to the respective section, if there is a good one, such as
> https://gcc.gnu.org/onlinedocs/gcc/ARM-Pragmas.html#ARM-Pragmas )?

Good idea, I'll send a patch to mention the link.
The relevant one is:
https://gcc.gnu.org/onlinedocs/gcc/AArch64-Function-Attributes.html#AArch64-Function-Attributes

Thanks,
Kyrill

>
> Gerald

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

[Kyrill Tkachov]

[-- Attachment #1: Type: text/plain, Size: 1231 bytes --]


On 07/09/15 13:16, Kyrill Tkachov wrote:
> Hi Gerald,
>
> On 07/09/15 12:31, Gerald Pfeifer wrote:
>> On Wed, 2 Sep 2015, Kyrill Tkachov wrote:
>>> 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.
>> I can follow your reasoning, and anyway the 48 hours are way over ;-),
>> just have you considered adding a reference to the documentation (as a
>> hyperlink to the respective section, if there is a good one, such as
>> https://gcc.gnu.org/onlinedocs/gcc/ARM-Pragmas.html#ARM-Pragmas )?
> Good idea, I'll send a patch to mention the link.
> The relevant one is:
> https://gcc.gnu.org/onlinedocs/gcc/AArch64-Function-Attributes.html#AArch64-Function-Attributes

And here is the patch.
I'll apply it in 48 hours if there's no objection.

Thanks,
Kyrill


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: wwwdocs-link.patch --]
[-- Type: text/x-patch; name=wwwdocs-link.patch, Size: 804 bytes --]

Index: htdocs/gcc-6/changes.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-6/changes.html,v
retrieving revision 1.26
diff -U 3 -r1.26 changes.html
--- htdocs/gcc-6/changes.html	4 Sep 2015 09:33:28 -0000	1.26
+++ htdocs/gcc-6/changes.html	7 Sep 2015 17:01:08 -0000
@@ -140,7 +140,8 @@
      </li>
      <li>
        The AArch64 port now supports target attributes and pragmas.  Please
-       refer to the documentation for details of available attributes and
+       refer to the <a href="https://gcc.gnu.org/onlinedocs/gcc/AArch64-Function-Attributes.html#AArch64-Function-Attributes">
+       documentation</a> for details of available attributes and
        pragmas as well as usage instructions.
      </li>
      <li>

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

[Gerald Pfeifer]

On Mon, 14 Sep 2015, Kyrill Tkachov wrote:
> And here is the patch.

Thanks.

> I'll apply it in 48 hours if there's no objection.

No need to wait. ;-)

Gerald

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

[Christian Bruel]

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 ?

many thanks

Christian

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

[Kyrill Tkachov]

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
>
>