public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH] wwwdocs: gcc-14: mark amdgcn fiji deprecated
@ 2023-10-19  9:49 Andrew Stubbs
  2023-10-19 10:07 ` Tobias Burnus
  0 siblings, 1 reply; 6+ messages in thread
From: Andrew Stubbs @ 2023-10-19  9:49 UTC (permalink / raw)
  To: gcc-patches; +Cc: tobias

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

OK to commit?

Andrew

[-- Attachment #2: 231019-wwwdocs-deprecate-fiji.patch --]
[-- Type: text/plain, Size: 808 bytes --]

gcc-14: mark amdgcn fiji deprecated


diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
index c817dde4..91ab8132 100644
--- a/htdocs/gcc-14/changes.html
+++ b/htdocs/gcc-14/changes.html
@@ -178,6 +178,16 @@ a work-in-progress.</p>
 
 <!-- <h3 id="aarch64">AArch64</h3> -->
 
+<h3 id="amdgcn">AMD Radeon (GCN)</h3>
+
+<ul>
+  <li>The Fiji device support is now deprecated and will be removed from a
+      future release.  The default compiler configuration no longer uses Fiji
+      as the default device, and no longer includes the Fiji libraries.  Both
+      can be restored by configuring with <code>--with-arch=fiji</code>.</li>
+  <li>The default device architecture is now <code>gfx900</code> (Vega).</li>
+</ul>
+
 <!-- <h3 id="arc">ARC</h3> -->
 
 <!-- <h3 id="arm">arm</h3> -->

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] wwwdocs: gcc-14: mark amdgcn fiji deprecated
  2023-10-19  9:49 [PATCH] wwwdocs: gcc-14: mark amdgcn fiji deprecated Andrew Stubbs
@ 2023-10-19 10:07 ` Tobias Burnus
  2023-10-20 12:58   ` Andrew Stubbs
  0 siblings, 1 reply; 6+ messages in thread
From: Tobias Burnus @ 2023-10-19 10:07 UTC (permalink / raw)
  To: Andrew Stubbs, gcc-patches; +Cc: Gerald Pfeifer

On 19.10.23 11:49, Andrew Stubbs wrote:
> OK to commit?

(I think as maintainer you don't need approval - but of course comments
by others can be helpful; I hope mine are. Additionally, Gerald (CCed)
helps with keeping the webpages in good shape (thanks!).)


> gcc-14: mark amdgcn fiji deprecated
> diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
> index c817dde4..91ab8132 100644
> --- a/htdocs/gcc-14/changes.html
> +++ b/htdocs/gcc-14/changes.html
> @@ -178,6 +178,16 @@ a work-in-progress.</p>
>
>   <!-- <h3 id="aarch64">AArch64</h3> -->
>
> +<h3 id="amdgcn">AMD Radeon (GCN)</h3>
> +
> +<ul>
> +  <li>The Fiji device support is now deprecated and will be removed from a
> +      future release.  The default compiler configuration no longer uses Fiji
> +      as the default device, and no longer includes the Fiji libraries.  Both
> +      can be restored by configuring with <code>--with-arch=fiji</code>.</li>
> +  <li>The default device architecture is now <code>gfx900</code> (Vega).</li>
> +</ul>

Can you add <a href="...">...</a> around the "--with-arch=fiji"? Linking to

https://gcc.gnu.org/install/specific.html#amdgcn-x-amdhsa

I think that page is helpful (once the cron job has updated that page).

Additionally, I wonder whether "Fiji" should be changed to "Fiji
(gfx803)" in the first line and whether the  "," should be removed in
"The ... configuration ... , and no longer includes".

Thanks,

Tobias


> +
>   <!-- <h3 id="arc">ARC</h3> -->
>
>   <!-- <h3 id="arm">arm</h3> -->
-----------------
Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht München, HRB 106955

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] wwwdocs: gcc-14: mark amdgcn fiji deprecated
  2023-10-19 10:07 ` Tobias Burnus
@ 2023-10-20 12:58   ` Andrew Stubbs
  2023-10-20 13:09     ` Tobias Burnus
  2023-10-22 12:24     ` Gerald Pfeifer
  0 siblings, 2 replies; 6+ messages in thread
From: Andrew Stubbs @ 2023-10-20 12:58 UTC (permalink / raw)
  To: Tobias Burnus, gcc-patches; +Cc: Gerald Pfeifer

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

On 19/10/2023 11:07, Tobias Burnus wrote:
> On 19.10.23 11:49, Andrew Stubbs wrote:
>> OK to commit?
> 
> (I think as maintainer you don't need approval - but of course comments
> by others can be helpful; I hope mine are. Additionally, Gerald (CCed)
> helps with keeping the webpages in good shape (thanks!).)
> 
> 
>> gcc-14: mark amdgcn fiji deprecated
>> diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
>> index c817dde4..91ab8132 100644
>> --- a/htdocs/gcc-14/changes.html
>> +++ b/htdocs/gcc-14/changes.html
>> @@ -178,6 +178,16 @@ a work-in-progress.</p>
>>
>>   <!-- <h3 id="aarch64">AArch64</h3> -->
>>
>> +<h3 id="amdgcn">AMD Radeon (GCN)</h3>
>> +
>> +<ul>
>> +  <li>The Fiji device support is now deprecated and will be removed 
>> from a
>> +      future release.  The default compiler configuration no longer 
>> uses Fiji
>> +      as the default device, and no longer includes the Fiji 
>> libraries.  Both
>> +      can be restored by configuring with 
>> <code>--with-arch=fiji</code>.</li>
>> +  <li>The default device architecture is now <code>gfx900</code> 
>> (Vega).</li>
>> +</ul>
> 
> Can you add <a href="...">...</a> around the "--with-arch=fiji"? Linking to
> 
> https://gcc.gnu.org/install/specific.html#amdgcn-x-amdhsa
> 
> I think that page is helpful (once the cron job has updated that page).
> 
> Additionally, I wonder whether "Fiji" should be changed to "Fiji
> (gfx803)" in the first line and whether the  "," should be removed in
> "The ... configuration ... , and no longer includes".

Fair enough, how's this version? (I like the comma, even if it is optional.)

Andrew

[-- Attachment #2: 231020-wwwdocs-deprecate-fiji.patch --]
[-- Type: text/plain, Size: 912 bytes --]

gcc-14: mark amdgcn fiji deprecated


diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
index c817dde4..23694310 100644
--- a/htdocs/gcc-14/changes.html
+++ b/htdocs/gcc-14/changes.html
@@ -178,6 +178,21 @@ a work-in-progress.</p>
 
 <!-- <h3 id="aarch64">AArch64</h3> -->
 
+<h3 id="amdgcn">AMD Radeon (GCN)</h3>
+
+<ul>
+  <li>
+    The Fiji (gfx803) device support is now deprecated and will be removed from
+    a future release.  The default compiler configuration no longer uses Fiji
+    as the default device, and no longer includes the Fiji libraries.  Both can
+    be restored by configuring with
+    <a href="https://gcc.gnu.org/install/specific.html#amdgcn-x-amdhsa">
+      <code>--with-arch=fiji</code>
+    </a>.
+  </li>
+  <li>The default device architecture is now <code>gfx900</code> (Vega).</li>
+</ul>
+
 <!-- <h3 id="arc">ARC</h3> -->
 
 <!-- <h3 id="arm">arm</h3> -->

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] wwwdocs: gcc-14: mark amdgcn fiji deprecated
  2023-10-20 12:58   ` Andrew Stubbs
@ 2023-10-20 13:09     ` Tobias Burnus
  2023-10-22 12:24     ` Gerald Pfeifer
  1 sibling, 0 replies; 6+ messages in thread
From: Tobias Burnus @ 2023-10-20 13:09 UTC (permalink / raw)
  To: Andrew Stubbs, Tobias Burnus, gcc-patches; +Cc: Gerald Pfeifer

On 20.10.23 14:58, Andrew Stubbs wrote:
> Fair enough, how's this version? (I like the comma, even if it is
> optional.)

LGTM – for what it is worth.

Thanks!

Tobias

-----------------
Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht München, HRB 106955

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] wwwdocs: gcc-14: mark amdgcn fiji deprecated
  2023-10-20 12:58   ` Andrew Stubbs
  2023-10-20 13:09     ` Tobias Burnus
@ 2023-10-22 12:24     ` Gerald Pfeifer
  2023-10-27 12:54       ` Andrew Stubbs
  1 sibling, 1 reply; 6+ messages in thread
From: Gerald Pfeifer @ 2023-10-22 12:24 UTC (permalink / raw)
  To: Andrew Stubbs; +Cc: Tobias Burnus, gcc-patches

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

Hi Andrew,

On Fri, 20 Oct 2023, Andrew Stubbs wrote:
>>  Additionally, I wonder whether "Fiji" should be changed to "Fiji
>> (gfx803)" in the first line and whether the  "," should be removed in
>> "The ... configuration ... , and no longer includes".
> Fair enough, how's this version? (I like the comma, even if it is optional.)

it's definitely fine. I do have a recommendation and a question, though 
feel free to go about them as you prefer.

+  <li>The default device architecture is now <code>gfx900</code> (Vega).</li>

How about starting with this as the first sub-item, as a "positive", 
then follow with the deprecation?

+<ul>
+  <li>
+    The Fiji (gfx803) device support is now deprecated and will be removed from

Could this be "Fiji (gfx803) device support" without the article?

Gerald

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] wwwdocs: gcc-14: mark amdgcn fiji deprecated
  2023-10-22 12:24     ` Gerald Pfeifer
@ 2023-10-27 12:54       ` Andrew Stubbs
  0 siblings, 0 replies; 6+ messages in thread
From: Andrew Stubbs @ 2023-10-27 12:54 UTC (permalink / raw)
  To: Gerald Pfeifer; +Cc: Tobias Burnus, gcc-patches

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

On 22/10/2023 13:24, Gerald Pfeifer wrote:
> Hi Andrew,
> 
> On Fri, 20 Oct 2023, Andrew Stubbs wrote:
>>>   Additionally, I wonder whether "Fiji" should be changed to "Fiji
>>> (gfx803)" in the first line and whether the  "," should be removed in
>>> "The ... configuration ... , and no longer includes".
>> Fair enough, how's this version? (I like the comma, even if it is optional.)
> 
> it's definitely fine. I do have a recommendation and a question, though
> feel free to go about them as you prefer.
> 
> +  <li>The default device architecture is now <code>gfx900</code> (Vega).</li>
> 
> How about starting with this as the first sub-item, as a "positive",
> then follow with the deprecation?
> 
> +<ul>
> +  <li>
> +    The Fiji (gfx803) device support is now deprecated and will be removed from
> 
> Could this be "Fiji (gfx803) device support" without the article?

Thank you for your suggestions; I have committed the attached.

Andrew

[-- Attachment #2: 231027-wwwdocs-deprecate-fiji.patch --]
[-- Type: text/plain, Size: 908 bytes --]

gcc-14: mark amdgcn fiji deprecated


diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
index c817dde4..a20499e9 100644
--- a/htdocs/gcc-14/changes.html
+++ b/htdocs/gcc-14/changes.html
@@ -178,6 +178,21 @@ a work-in-progress.</p>
 
 <!-- <h3 id="aarch64">AArch64</h3> -->
 
+<h3 id="amdgcn">AMD Radeon (GCN)</h3>
+
+<ul>
+  <li>The default device architecture is now <code>gfx900</code> (Vega).</li>
+  <li>
+    Fiji (gfx803) device support is now deprecated and will be removed from
+    a future release.  The default compiler configuration no longer uses Fiji
+    as the default device, and no longer includes the Fiji libraries.  Both can
+    be restored by configuring with
+    <a href="https://gcc.gnu.org/install/specific.html#amdgcn-x-amdhsa">
+      <code>--with-arch=fiji</code>
+    </a>.
+  </li>
+</ul>
+
 <!-- <h3 id="arc">ARC</h3> -->
 
 <!-- <h3 id="arm">arm</h3> -->

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2023-10-27 12:54 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-10-19  9:49 [PATCH] wwwdocs: gcc-14: mark amdgcn fiji deprecated Andrew Stubbs
2023-10-19 10:07 ` Tobias Burnus
2023-10-20 12:58   ` Andrew Stubbs
2023-10-20 13:09     ` Tobias Burnus
2023-10-22 12:24     ` Gerald Pfeifer
2023-10-27 12:54       ` Andrew Stubbs

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).