[3/3] Enable full libgfortran library for AMD GCN

Message ID a7d088ad-3a0e-d18e-5577-ee7ca0803aa3@codesourcery.com
State New
Headers show
Series
  • Enable full libgfortran for AMD GCN
Related show

Commit Message

Andrew Stubbs June 7, 2019, 2:40 p.m.
This patch basically reverts the previous patch to put AMD GCN in 
"minimal" mode.

OK to commit?

-- 
Andrew Stubbs
Mentor Graphics / CodeSourcery

Comments

Janne Blomqvist June 14, 2019, 4:17 p.m. | #1
On Fri, Jun 7, 2019 at 5:41 PM Andrew Stubbs <ams@codesourcery.com> wrote:
>

> This patch basically reverts the previous patch to put AMD GCN in

> "minimal" mode.


This is Ok, provided the maintainers for the areas touched by the
other 2 patches in this series Ok those.

Thanks for improving Fortran offloading (given the recent Frontier
supercomputer announcement, I hope we will receive more work in this
area in the future).


-- 
Janne Blomqvist
Jeff Law June 19, 2019, 4:05 p.m. | #2
On 6/7/19 8:40 AM, Andrew Stubbs wrote:
> This patch basically reverts the previous patch to put AMD GCN in

> "minimal" mode.

> 

> OK to commit?

OK
jeff

Patch

Enable full GFortran library for AMD GCN

2019-06-07  Kwok Cheung Yeung  <kcy@codesourcery.com>
            Andrew Stubbs  <ams@codesourcery.com>

	libgfortran/
	* configure: Regenerate.
	* configure.ac (LIBGFOR_MINIMAL): Do not use on AMD GCN.

diff --git a/libgfortran/configure b/libgfortran/configure
index 60867b93d0e..15c9307f9d5 100755
--- a/libgfortran/configure
+++ b/libgfortran/configure
@@ -6179,8 +6179,7 @@  fi
 # * C library support for other features such as signal, environment
 #   variables, time functions
 
- if test "x${target_cpu}" = xnvptx \
-				 || test "x${target_cpu}" = xamdgcn; then
+ if test "x${target_cpu}" = xnvptx; then
   LIBGFOR_MINIMAL_TRUE=
   LIBGFOR_MINIMAL_FALSE='#'
 else
diff --git a/libgfortran/configure.ac b/libgfortran/configure.ac
index 7cfce28ab69..8dcc8606bd4 100644
--- a/libgfortran/configure.ac
+++ b/libgfortran/configure.ac
@@ -205,8 +205,7 @@  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 \
-				 || test "x${target_cpu}" = xamdgcn])
+AM_CONDITIONAL(LIBGFOR_MINIMAL, [test "x${target_cpu}" = xnvptx])
 
 # Figure out whether the compiler supports "-ffunction-sections -fdata-sections",
 # similarly to how libstdc++ does it