Discussion:
[PATCH 17/25] Fix Fortran STOP.
Toon Moene
2018-09-05 16:55:36 UTC
Permalink
-------- Forwarded Message --------
Subject: [PATCH 17/25] Fix Fortran STOP.
Date: Wed, 5 Sep 2018 12:51:18 +0100
From: ***@codesourcery.com
To: gcc-***@gcc.gnu.org


The minimal libgfortran setup was created for NVPTX, but will also be
used by
AMD GCN.

This patch simply removes an assumption that NVPTX is the only user.
Specifically, NVPTX exit is broken, but AMD GCN exit works just fine.

2018-09-05 Andrew Stubbs <***@codesourcery.com>

libgfortran/
* runtime/minimal.c (exit): Only work around nvptx bugs on nvptx.
---
libgfortran/runtime/minimal.c | 2 ++
1 file changed, 2 insertions(+)
Bernhard Reutner-Fischer
2018-09-05 17:06:32 UTC
Permalink
On Wed, 5 Sep 2018 at 18:55, Toon Moene <***@moene.org> wrote:
Looks ok and obvious but i cannot approve it.
cheers,
Post by Toon Moene
-------- Forwarded Message --------
Subject: [PATCH 17/25] Fix Fortran STOP.
Date: Wed, 5 Sep 2018 12:51:18 +0100
The minimal libgfortran setup was created for NVPTX, but will also be
used by
AMD GCN.
This patch simply removes an assumption that NVPTX is the only user.
Specifically, NVPTX exit is broken, but AMD GCN exit works just fine.
libgfortran/
* runtime/minimal.c (exit): Only work around nvptx bugs on nvptx.
---
libgfortran/runtime/minimal.c | 2 ++
1 file changed, 2 insertions(+)
Janne Blomqvist
2018-09-05 18:09:16 UTC
Permalink
Same, please send fortran patches to the fortran list as well!
Post by Toon Moene
-------- Forwarded Message --------
Subject: [PATCH 17/25] Fix Fortran STOP.
Date: Wed, 5 Sep 2018 12:51:18 +0100
The minimal libgfortran setup was created for NVPTX, but will also be
used by
AMD GCN.
This patch simply removes an assumption that NVPTX is the only user.
Specifically, NVPTX exit is broken, but AMD GCN exit works just fine.
libgfortran/
* runtime/minimal.c (exit): Only work around nvptx bugs on nvptx.
---
libgfortran/runtime/minimal.c | 2 ++
1 file changed, 2 insertions(+)
Ok, thanks.
--
Janne Blomqvist
Andrew Stubbs
2018-09-12 13:56:19 UTC
Permalink
Post by Janne Blomqvist
Ok, thanks.
Committed, thanks.

Andrew

Loading...