Fwd: [PATCH 19/25] GCN libgfortran.

Fwd: [PATCH 19/25] GCN libgfortran.

[Toon Moene]

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




-------- Forwarded Message --------
Subject: [PATCH 19/25] GCN libgfortran.
Date: Wed, 5 Sep 2018 12:51:20 +0100
From: ams@codesourcery.com
To: gcc-patches@gcc.gnu.org


This patch contains the GCN port of libgfortran.  We use the minimal
configuration created for NVPTX.  That's all that's required, besides the
target-independent bug fixes posted already.

2018-09-05  Andrew Stubbs  <ams@codesourcery.com>
	    Kwok Cheung Yeung  <kcy@codesourcery.com>
	    Julian Brown  <julian@codesourcery.com>
	    Tom de Vries  <tom@codesourcery.com>

	libgfortran/
	* configure.ac: Use minimal mode for amdgcn.
	* configure: Regenerate.
---
  libgfortran/configure    | 7 ++++---
  libgfortran/configure.ac | 3 ++-
  2 files changed, 6 insertions(+), 4 deletions(-)



[-- Attachment #2: 0019-GCN-libgfortran.patch --]
[-- Type: text/x-patch, Size: 1601 bytes --]

diff --git a/libgfortran/configure b/libgfortran/configure
index a583b67..fd8b697 100755
--- a/libgfortran/configure
+++ b/libgfortran/configure
@@ -5994,7 +5994,8 @@ fi
 # * C library support for other features such as signal, environment
 #   variables, time functions
 
- if test "x${target_cpu}" = xnvptx; then
+ if test "x${target_cpu}" = xnvptx \
+				 || test "x${target_cpu}" = xamdgcn; then
   LIBGFOR_MINIMAL_TRUE=
   LIBGFOR_MINIMAL_FALSE='#'
 else
@@ -12514,7 +12515,7 @@ else
   lt_dlunknown=0; lt_dlno_uscore=1; lt_dlneed_uscore=2
   lt_status=$lt_dlunknown
   cat > conftest.$ac_ext <<_LT_EOF
-#line 12517 "configure"
+#line 12518 "configure"
 #include "confdefs.h"
 
 #if HAVE_DLFCN_H
@@ -12620,7 +12621,7 @@ else
   lt_dlunknown=0; lt_dlno_uscore=1; lt_dlneed_uscore=2
   lt_status=$lt_dlunknown
   cat > conftest.$ac_ext <<_LT_EOF
-#line 12623 "configure"
+#line 12624 "configure"
 #include "confdefs.h"
 
 #if HAVE_DLFCN_H
diff --git a/libgfortran/configure.ac b/libgfortran/configure.ac
index 05952aa..11b629d 100644
--- a/libgfortran/configure.ac
+++ b/libgfortran/configure.ac
@@ -206,7 +206,8 @@ AM_CONDITIONAL(LIBGFOR_USE_SYMVER_SUN, [test "x$gfortran_use_symver" = xsun])
 # * C library support for other features such as signal, environment
 #   variables, time functions
 
-AM_CONDITIONAL(LIBGFOR_MINIMAL, [test "x${target_cpu}" = xnvptx])
+AM_CONDITIONAL(LIBGFOR_MINIMAL, [test "x${target_cpu}" = xnvptx \
+				 || test "x${target_cpu}" = xamdgcn])
 
 # Figure out whether the compiler supports "-ffunction-sections -fdata-sections",
 # similarly to how libstdc++ does it

Re: [PATCH 19/25] GCN libgfortran.

[Bernhard Reutner-Fischer]

On Wed, 5 Sep 2018 at 18:56, Toon Moene <toon@moene.org> wrote:
>
>

Excess whitespace but looks sane otherwise.
AM_CONDITIONAL(LIBGFOR_MINIMAL, [
  test "x${target_cpu}" = xnvptx \
  || test "x${target_cpu}" = xamdgcn])

would be better IMHO.

cheers,
>
>
> -------- Forwarded Message --------
> Subject: [PATCH 19/25] GCN libgfortran.
> Date: Wed, 5 Sep 2018 12:51:20 +0100
> From: ams@codesourcery.com
> To: gcc-patches@gcc.gnu.org
>
>
> This patch contains the GCN port of libgfortran.  We use the minimal
> configuration created for NVPTX.  That's all that's required, besides the
> target-independent bug fixes posted already.
>
> 2018-09-05  Andrew Stubbs  <ams@codesourcery.com>
>             Kwok Cheung Yeung  <kcy@codesourcery.com>
>             Julian Brown  <julian@codesourcery.com>
>             Tom de Vries  <tom@codesourcery.com>
>
>         libgfortran/
>         * configure.ac: Use minimal mode for amdgcn.
>         * configure: Regenerate.
> ---
>   libgfortran/configure    | 7 ++++---
>   libgfortran/configure.ac | 3 ++-
>   2 files changed, 6 insertions(+), 4 deletions(-)
>
>

Re: [PATCH 19/25] GCN libgfortran.

[Janne Blomqvist]

Please send fortran patches to the fortran list as well!

On Wed, Sep 5, 2018 at 7:56 PM Toon Moene <toon@moene.org> wrote:

>
>
>
> -------- Forwarded Message --------
> Subject: [PATCH 19/25] GCN libgfortran.
> Date: Wed, 5 Sep 2018 12:51:20 +0100
> From: ams@codesourcery.com
> To: gcc-patches@gcc.gnu.org
>
>
> This patch contains the GCN port of libgfortran.  We use the minimal
> configuration created for NVPTX.  That's all that's required, besides the
> target-independent bug fixes posted already.
>
> 2018-09-05  Andrew Stubbs  <ams@codesourcery.com>
>             Kwok Cheung Yeung  <kcy@codesourcery.com>
>             Julian Brown  <julian@codesourcery.com>
>             Tom de Vries  <tom@codesourcery.com>
>
>         libgfortran/
>         * configure.ac: Use minimal mode for amdgcn.
>         * configure: Regenerate.
> ---
>   libgfortran/configure    | 7 ++++---
>   libgfortran/configure.ac | 3 ++-
>   2 files changed, 6 insertions(+), 4 deletions(-)
>
>
>
Ok!

-- 
Janne Blomqvist

Re: [PATCH 19/25] GCN libgfortran.

[Andrew Stubbs]

On 05/09/18 19:14, Janne Blomqvist wrote:
> Please send fortran patches to the fortran list as well!

Apologies, I was not aware of this.

> Ok!

Thanks, I will commit when the rest of the port is approved.

Andrew