[patch][GCN] install.texi: Update GCN entry - @uref and LLVM version remark

[patch][GCN] install.texi: Update GCN entry - @uref and LLVM version remark

[Tobias Burnus]

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

Stumbled over this.

I wondered whether we should recommend newlib >= 12e3bac3c (31st Oct 2023), but
given that gfx1030 is not yet supported, I decided against it. We can revisit
this once newlib 4.4 is available.

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

Comments?

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

[-- Attachment #2: gcn-install.diff --]
[-- Type: text/x-patch, Size: 1162 bytes --]

install.texi: Update GCN entry - @uref and LLVM version remark

gcc/ChangeLog:

	* doc/install.texi (amdgcn-*-amdhsa): Fix URL to ROCm;
	change 'in the future' to 'in LLVM 18'.

 gcc/doc/install.texi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gcc/doc/install.texi b/gcc/doc/install.texi
index f995b775f83..b515cd1469f 100644
--- a/gcc/doc/install.texi
+++ b/gcc/doc/install.texi
@@ -3991,7 +3991,7 @@ Instead of GNU Binutils, you will need to install LLVM 13.0.1, or later, and cop
 Use Newlib (4.3.0 or newer).
 
 To run the binaries, install the HSA Runtime from the
-@uref{https://rocm.github.io,,ROCm Platform}, and use
+@uref{https://rocm.docs.amd.com/,,ROCm Platform}, and use
 @file{libexec/gcc/amdhsa-amdhsa/@var{version}/gcn-run} to launch them
 on the GPU.
 
@@ -3999,7 +3999,7 @@ To enable support for GCN3 Fiji devices (gfx803), GCC has to be configured with
 @option{--with-arch=@code{fiji}} or
 @option{--with-multilib-list=@code{fiji},...}.  Note that support for Fiji
 devices has been removed in ROCm 4.0 and support in LLVM is deprecated and will
-be removed in the future.
+be removed in LLVM 18.
 
 @html
 <hr />

Re: [patch][GCN] install.texi: Update GCN entry - @uref and LLVM version remark

[Andrew Stubbs]

On 24/11/2023 16:07, Tobias Burnus wrote:
> Stumbled over this.
> 
> I wondered whether we should recommend newlib >= 12e3bac3c (31st Oct 
> 2023), but
> given that gfx1030 is not yet supported, I decided against it. We can 
> revisit
> this once newlib 4.4 is available.

This makes sense to me. We can maybe add that detail to the Wiki (which 
I also haven't got around to updating).

> Current version is: 
> https://gcc.gnu.org/install/specific.html#amdgcn-x-amdhsa>> 
> Comments?

I think the patch is accurate.

OK.

Andrew

Re: [patch][GCN] install.texi: Update GCN entry - @uref and LLVM version remark

[Gerald Pfeifer]

On Fri, 24 Nov 2023, Tobias Burnus wrote:
> Stumbled over this.

Same here. :-)

> Comments?

Thank you for fixing this.

Gerald