diff mbox series

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

Message ID be2d7503-cf71-47ea-9fb5-5f069e3cdd9c@codesourcery.com
State New
Headers show
Series [GCN] install.texi: Update GCN entry - @uref and LLVM version remark | expand

Commit Message

Tobias Burnus Nov. 24, 2023, 4:07 p.m. UTC 
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

Comments

Andrew Stubbs Nov. 24, 2023, 6 p.m. UTC | #1 
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
Gerald Pfeifer Nov. 25, 2023, 11:25 a.m. UTC | #2 
On Fri, 24 Nov 2023, Tobias Burnus wrote:
> Stumbled over this.

Same here. :-)

> Comments?

Thank you for fixing this.

Gerald
diff mbox series

Patch

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