[19/25] GCN libgfortran.

Message ID f522064c707f4f3c03735790babe0d384c0cf7ea.1536144068.git.ams@codesourcery.com
State Superseded
Headers show
Series
  • AMD GCN Port
Related show

Commit Message

Andrew Stubbs Sept. 5, 2018, 11:51 a.m.
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(-)

Comments

Janne Blomqvist Sept. 5, 2018, 6:14 p.m. | #1
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
Andrew Stubbs Sept. 6, 2018, 12:37 p.m. | #2
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
Jeff Law Sept. 11, 2018, 10:47 p.m. | #3
On 9/5/18 5:51 AM, ams@codesourcery.com wrote:
> 

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

This is OK once the core port has been accepted.

jeff

Patch

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