* driver: Forward '-lgfortran', '-lm' to offloading compilation
@ 2023-06-05 12:25 Thomas Schwinge
2023-06-13 10:44 ` [ping] " Thomas Schwinge
0 siblings, 1 reply; 4+ messages in thread
From: Thomas Schwinge @ 2023-06-05 12:25 UTC (permalink / raw)
To: gcc-patches, Joseph Myers, Jakub Jelinek, Tobias Burnus; +Cc: fortran
[-- Attachment #1: Type: text/plain, Size: 443 bytes --]
Hi!
OK to push the attached
"driver: Forward '-lgfortran', '-lm' to offloading compilation"?
(We didn't have a PR open for that, or did we?)
Grüße
Thomas
-----------------
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
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-driver-Forward-lgfortran-lm-to-offloading-compilatio.patch --]
[-- Type: text/x-diff, Size: 9867 bytes --]
From 5d3cb866cad3bbcf47c5e66825e5710e86cc017e Mon Sep 17 00:00:00 2001
From: Thomas Schwinge <thomas@codesourcery.com>
Date: Mon, 5 Jun 2023 11:26:37 +0200
Subject: [PATCH] driver: Forward '-lgfortran', '-lm' to offloading compilation
..., so that users don't manually need to specify
'-foffload-options=-lgfortran', '-foffload-options=-lm' in addition to
'-lgfortran', '-lm' (specified manually, or implicitly by the driver).
gcc/
* gcc.cc (driver_handle_option): Forward host '-lgfortran', '-lm'
to offloading compilation.
* config/gcn/mkoffload.cc (main): Adjust.
* config/nvptx/mkoffload.cc (main): Likewise.
* doc/invoke.texi (foffload-options): Update example.
libgomp/
* testsuite/libgomp.fortran/fortran.exp (lang_link_flags): Don't
set.
* testsuite/libgomp.oacc-fortran/fortran.exp (lang_link_flags):
Likewise.
* testsuite/libgomp.c/simd-math-1.c: Remove
'-foffload-options=-lm'.
* testsuite/libgomp.fortran/fortran-torture_execute_math.f90:
Likewise.
* testsuite/libgomp.oacc-fortran/fortran-torture_execute_math.f90:
Likewise.
---
gcc/config/gcn/mkoffload.cc | 12 ++++
gcc/config/nvptx/mkoffload.cc | 12 ++++
gcc/doc/invoke.texi | 5 +-
gcc/gcc.cc | 56 +++++++++++++++++++
libgomp/testsuite/libgomp.c/simd-math-1.c | 1 -
.../fortran-torture_execute_math.f90 | 1 -
libgomp/testsuite/libgomp.fortran/fortran.exp | 2 -
.../fortran-torture_execute_math.f90 | 1 -
.../libgomp.oacc-fortran/fortran.exp | 2 -
9 files changed, 82 insertions(+), 10 deletions(-)
diff --git a/gcc/config/gcn/mkoffload.cc b/gcc/config/gcn/mkoffload.cc
index 988c12318fd..8b608bf024e 100644
--- a/gcc/config/gcn/mkoffload.cc
+++ b/gcc/config/gcn/mkoffload.cc
@@ -946,6 +946,18 @@ main (int argc, char **argv)
else if (startswith (argv[i], STR))
gcn_stack_size = atoi (argv[i] + strlen (STR));
#undef STR
+ /* Translate host into offloading libraries. */
+ else if (strcmp (argv[i], "-l_GCC_gfortran") == 0
+ || strcmp (argv[i], "-l_GCC_m") == 0)
+ {
+ /* Elide '_GCC_'. */
+ size_t i_dst = strlen ("-l");
+ size_t i_src = strlen ("-l_GCC_");
+ char c;
+ do
+ c = argv[i][i_dst++] = argv[i][i_src++];
+ while (c != '\0');
+ }
}
if (!(fopenacc ^ fopenmp))
diff --git a/gcc/config/nvptx/mkoffload.cc b/gcc/config/nvptx/mkoffload.cc
index 6cdea45cffe..aaea9fb320d 100644
--- a/gcc/config/nvptx/mkoffload.cc
+++ b/gcc/config/nvptx/mkoffload.cc
@@ -649,6 +649,18 @@ main (int argc, char **argv)
else if (strcmp (argv[i], "-dumpbase") == 0
&& i + 1 < argc)
dumppfx = argv[++i];
+ /* Translate host into offloading libraries. */
+ else if (strcmp (argv[i], "-l_GCC_gfortran") == 0
+ || strcmp (argv[i], "-l_GCC_m") == 0)
+ {
+ /* Elide '_GCC_'. */
+ size_t i_dst = strlen ("-l");
+ size_t i_src = strlen ("-l_GCC_");
+ char c;
+ do
+ c = argv[i][i_dst++] = argv[i][i_src++];
+ while (c != '\0');
+ }
}
if (!(fopenacc ^ fopenmp))
fatal_error (input_location, "either %<-fopenacc%> or %<-fopenmp%> "
diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
index d2d639c92d4..7b3a2a74459 100644
--- a/gcc/doc/invoke.texi
+++ b/gcc/doc/invoke.texi
@@ -2716,9 +2716,8 @@ the @option{-foffload-options=@var{target-list}=@var{options}} form. The
Typical command lines are
@smallexample
--foffload-options=-lgfortran -foffload-options=-lm
--foffload-options="-lgfortran -lm" -foffload-options=nvptx-none=-latomic
--foffload-options=amdgcn-amdhsa=-march=gfx906 -foffload-options=-lm
+-foffload-options='-fno-math-errno -ffinite-math-only' -foffload-options=nvptx-none=-latomic
+-foffload-options=amdgcn-amdhsa=-march=gfx906 -foffload-options=-O3
@end smallexample
@opindex fopenacc
diff --git a/gcc/gcc.cc b/gcc/gcc.cc
index 2ccca00d603..15995206856 100644
--- a/gcc/gcc.cc
+++ b/gcc/gcc.cc
@@ -47,6 +47,9 @@ compilation is specified by a string called a "spec". */
#include "opts-jobserver.h"
#include "common/common-target.h"
+#ifndef MATH_LIBRARY
+#define MATH_LIBRARY "m"
+#endif
\f
/* Manage the manipulation of env vars.
@@ -4117,6 +4120,48 @@ next_item:
}
}
+/* Forward certain options to offloading compilation. */
+
+static void
+forward_offload_option (size_t opt_index, const char *arg, bool validated)
+{
+ switch (opt_index)
+ {
+ case OPT_l:
+ /* Use a '_GCC_' prefix and standard name ('-l_GCC_m' irrespective of the
+ host's 'MATH_LIBRARY', for example), so that the 'mkoffload's can tell
+ this has been synthesized here, and translate/drop as necessary. */
+ /* Note that certain libraries ('-lc', '-lgcc', '-lgomp', for example)
+ are injected by default in offloading compilation, and therefore not
+ forwarded here. */
+ /* GCC libraries. */
+ if (/* '-lgfortran' */ strcmp (arg, "gfortran") == 0 )
+ save_switch (concat ("-foffload-options=-l_GCC_", arg, NULL),
+ 0, NULL, validated, true);
+ /* Other libraries. */
+ else
+ {
+ /* The case will need special consideration where on the host
+ '!need_math', but for offloading compilation still need
+ '-foffload-options=-l_GCC_m'. The problem is that we don't get
+ here anything like '-lm', because it's not synthesized in
+ 'gcc/fortran/gfortranspec.cc:lang_specific_driver', for example.
+ Generally synthesizing '-foffload-options=-l_GCC_m' etc. in the
+ language specific drivers is non-trivial, needs very careful
+ review of their options handling. However, this issue is not
+ actually relevant for the current set of supported host/offloading
+ configurations. */
+ int need_math = (MATH_LIBRARY[0] != '\0');
+ if (/* '-lm' */ (need_math && strcmp (arg, MATH_LIBRARY) == 0))
+ save_switch ("-foffload-options=-l_GCC_m",
+ 0, NULL, validated, true);
+ }
+ break;
+ default:
+ gcc_unreachable ();
+ }
+}
+
/* Handle a driver option; arguments and return value as for
handle_option. */
@@ -4375,6 +4420,17 @@ driver_handle_option (struct gcc_options *opts,
/* POSIX allows separation of -l and the lib arg; canonicalize
by concatenating -l with its arg */
add_infile (concat ("-l", arg, NULL), "*");
+
+ /* Forward to offloading compilation '-l[...]' flags for standard,
+ well-known libraries. */
+ /* Doing this processing here means that we don't get to see libraries
+ injected via specs, such as '-lquadmath' injected via
+ '[build]/[target]/libgfortran/libgfortran.spec'. However, this issue
+ is not actually relevant for the current set of host/offloading
+ configurations. */
+ if (ENABLE_OFFLOADING)
+ forward_offload_option (opt_index, arg, validated);
+
do_save = false;
break;
diff --git a/libgomp/testsuite/libgomp.c/simd-math-1.c b/libgomp/testsuite/libgomp.c/simd-math-1.c
index 947bf606e36..dd2077cc597 100644
--- a/libgomp/testsuite/libgomp.c/simd-math-1.c
+++ b/libgomp/testsuite/libgomp.c/simd-math-1.c
@@ -4,7 +4,6 @@
/* { dg-do run } */
/* { dg-options "-O2 -ftree-vectorize -fno-math-errno" } */
/* { dg-additional-options -foffload-options=amdgcn-amdhsa=-mstack-size=3000000 { target offload_target_amdgcn } } */
-/* { dg-additional-options -foffload-options=-lm } */
#undef PRINT_RESULT
#define VERBOSE 0
diff --git a/libgomp/testsuite/libgomp.fortran/fortran-torture_execute_math.f90 b/libgomp/testsuite/libgomp.fortran/fortran-torture_execute_math.f90
index 3348a0bb3ad..2d0caa6b84a 100644
--- a/libgomp/testsuite/libgomp.fortran/fortran-torture_execute_math.f90
+++ b/libgomp/testsuite/libgomp.fortran/fortran-torture_execute_math.f90
@@ -1,4 +1,3 @@
! { dg-do run }
-! { dg-additional-options -foffload-options=-lm }
include '../../../gcc/testsuite/gfortran.fortran-torture/execute/math.f90'
diff --git a/libgomp/testsuite/libgomp.fortran/fortran.exp b/libgomp/testsuite/libgomp.fortran/fortran.exp
index e69656bce23..32e4bb2af4e 100644
--- a/libgomp/testsuite/libgomp.fortran/fortran.exp
+++ b/libgomp/testsuite/libgomp.fortran/fortran.exp
@@ -28,7 +28,6 @@ if { $blddir != "" } {
set lang_source_re {^.*\.[fF](|90|95|03|08)$}
set lang_include_flags "-fintrinsic-modules-path=${blddir}"
}
-set lang_link_flags "-foffload=-lgfortran"
lappend ALWAYS_CFLAGS "compiler=$GFORTRAN_UNDER_TEST"
# Initialize dg.
@@ -63,7 +62,6 @@ if { $blddir != "" } {
unset libquadmath_library_path
unset lang_library_paths
}
-unset lang_link_flags
# All done.
dg-finish
diff --git a/libgomp/testsuite/libgomp.oacc-fortran/fortran-torture_execute_math.f90 b/libgomp/testsuite/libgomp.oacc-fortran/fortran-torture_execute_math.f90
index 1b2ac440762..edc940c4706 100644
--- a/libgomp/testsuite/libgomp.oacc-fortran/fortran-torture_execute_math.f90
+++ b/libgomp/testsuite/libgomp.oacc-fortran/fortran-torture_execute_math.f90
@@ -1,5 +1,4 @@
! { dg-do run }
!TODO { dg-prune-output {using 'vector_length \(32\)', ignoring 1} }
-! { dg-additional-options -foffload-options=-lm }
include '../../../gcc/testsuite/gfortran.fortran-torture/execute/math.f90'
diff --git a/libgomp/testsuite/libgomp.oacc-fortran/fortran.exp b/libgomp/testsuite/libgomp.oacc-fortran/fortran.exp
index 2e8aa2285fb..e5844ad00fa 100644
--- a/libgomp/testsuite/libgomp.oacc-fortran/fortran.exp
+++ b/libgomp/testsuite/libgomp.oacc-fortran/fortran.exp
@@ -30,7 +30,6 @@ if { $blddir != "" } {
set lang_source_re {^.*\.[fF](|90|95|03|08)$}
set lang_include_flags "-fintrinsic-modules-path=${blddir}"
}
-set lang_link_flags "-foffload=-lgfortran"
lappend ALWAYS_CFLAGS "compiler=$GFORTRAN_UNDER_TEST"
# Initialize dg.
@@ -113,7 +112,6 @@ if { $blddir != "" } {
unset libquadmath_library_path
unset lang_library_paths
}
-unset lang_link_flags
# All done.
dg-finish
--
2.34.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* [ping] driver: Forward '-lgfortran', '-lm' to offloading compilation
2023-06-05 12:25 driver: Forward '-lgfortran', '-lm' to offloading compilation Thomas Schwinge
@ 2023-06-13 10:44 ` Thomas Schwinge
2023-06-13 15:34 ` Joseph Myers
2023-06-14 10:13 ` Tobias Burnus
0 siblings, 2 replies; 4+ messages in thread
From: Thomas Schwinge @ 2023-06-13 10:44 UTC (permalink / raw)
To: gcc-patches, Joseph Myers, Jakub Jelinek, Tobias Burnus; +Cc: fortran
[-- Attachment #1: Type: text/plain, Size: 497 bytes --]
Hi!
On 2023-06-05T14:25:18+0200, I wrote:
> OK to push the attached
> "driver: Forward '-lgfortran', '-lm' to offloading compilation"?
> (We didn't have a PR open for that, or did we?)
Ping.
Grüße
Thomas
-----------------
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
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-driver-Forward-lgfortran-lm-to-offloading-compilatio.patch --]
[-- Type: text/x-diff, Size: 9867 bytes --]
From 5d3cb866cad3bbcf47c5e66825e5710e86cc017e Mon Sep 17 00:00:00 2001
From: Thomas Schwinge <thomas@codesourcery.com>
Date: Mon, 5 Jun 2023 11:26:37 +0200
Subject: [PATCH] driver: Forward '-lgfortran', '-lm' to offloading compilation
..., so that users don't manually need to specify
'-foffload-options=-lgfortran', '-foffload-options=-lm' in addition to
'-lgfortran', '-lm' (specified manually, or implicitly by the driver).
gcc/
* gcc.cc (driver_handle_option): Forward host '-lgfortran', '-lm'
to offloading compilation.
* config/gcn/mkoffload.cc (main): Adjust.
* config/nvptx/mkoffload.cc (main): Likewise.
* doc/invoke.texi (foffload-options): Update example.
libgomp/
* testsuite/libgomp.fortran/fortran.exp (lang_link_flags): Don't
set.
* testsuite/libgomp.oacc-fortran/fortran.exp (lang_link_flags):
Likewise.
* testsuite/libgomp.c/simd-math-1.c: Remove
'-foffload-options=-lm'.
* testsuite/libgomp.fortran/fortran-torture_execute_math.f90:
Likewise.
* testsuite/libgomp.oacc-fortran/fortran-torture_execute_math.f90:
Likewise.
---
gcc/config/gcn/mkoffload.cc | 12 ++++
gcc/config/nvptx/mkoffload.cc | 12 ++++
gcc/doc/invoke.texi | 5 +-
gcc/gcc.cc | 56 +++++++++++++++++++
libgomp/testsuite/libgomp.c/simd-math-1.c | 1 -
.../fortran-torture_execute_math.f90 | 1 -
libgomp/testsuite/libgomp.fortran/fortran.exp | 2 -
.../fortran-torture_execute_math.f90 | 1 -
.../libgomp.oacc-fortran/fortran.exp | 2 -
9 files changed, 82 insertions(+), 10 deletions(-)
diff --git a/gcc/config/gcn/mkoffload.cc b/gcc/config/gcn/mkoffload.cc
index 988c12318fd..8b608bf024e 100644
--- a/gcc/config/gcn/mkoffload.cc
+++ b/gcc/config/gcn/mkoffload.cc
@@ -946,6 +946,18 @@ main (int argc, char **argv)
else if (startswith (argv[i], STR))
gcn_stack_size = atoi (argv[i] + strlen (STR));
#undef STR
+ /* Translate host into offloading libraries. */
+ else if (strcmp (argv[i], "-l_GCC_gfortran") == 0
+ || strcmp (argv[i], "-l_GCC_m") == 0)
+ {
+ /* Elide '_GCC_'. */
+ size_t i_dst = strlen ("-l");
+ size_t i_src = strlen ("-l_GCC_");
+ char c;
+ do
+ c = argv[i][i_dst++] = argv[i][i_src++];
+ while (c != '\0');
+ }
}
if (!(fopenacc ^ fopenmp))
diff --git a/gcc/config/nvptx/mkoffload.cc b/gcc/config/nvptx/mkoffload.cc
index 6cdea45cffe..aaea9fb320d 100644
--- a/gcc/config/nvptx/mkoffload.cc
+++ b/gcc/config/nvptx/mkoffload.cc
@@ -649,6 +649,18 @@ main (int argc, char **argv)
else if (strcmp (argv[i], "-dumpbase") == 0
&& i + 1 < argc)
dumppfx = argv[++i];
+ /* Translate host into offloading libraries. */
+ else if (strcmp (argv[i], "-l_GCC_gfortran") == 0
+ || strcmp (argv[i], "-l_GCC_m") == 0)
+ {
+ /* Elide '_GCC_'. */
+ size_t i_dst = strlen ("-l");
+ size_t i_src = strlen ("-l_GCC_");
+ char c;
+ do
+ c = argv[i][i_dst++] = argv[i][i_src++];
+ while (c != '\0');
+ }
}
if (!(fopenacc ^ fopenmp))
fatal_error (input_location, "either %<-fopenacc%> or %<-fopenmp%> "
diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
index d2d639c92d4..7b3a2a74459 100644
--- a/gcc/doc/invoke.texi
+++ b/gcc/doc/invoke.texi
@@ -2716,9 +2716,8 @@ the @option{-foffload-options=@var{target-list}=@var{options}} form. The
Typical command lines are
@smallexample
--foffload-options=-lgfortran -foffload-options=-lm
--foffload-options="-lgfortran -lm" -foffload-options=nvptx-none=-latomic
--foffload-options=amdgcn-amdhsa=-march=gfx906 -foffload-options=-lm
+-foffload-options='-fno-math-errno -ffinite-math-only' -foffload-options=nvptx-none=-latomic
+-foffload-options=amdgcn-amdhsa=-march=gfx906 -foffload-options=-O3
@end smallexample
@opindex fopenacc
diff --git a/gcc/gcc.cc b/gcc/gcc.cc
index 2ccca00d603..15995206856 100644
--- a/gcc/gcc.cc
+++ b/gcc/gcc.cc
@@ -47,6 +47,9 @@ compilation is specified by a string called a "spec". */
#include "opts-jobserver.h"
#include "common/common-target.h"
+#ifndef MATH_LIBRARY
+#define MATH_LIBRARY "m"
+#endif
\f
/* Manage the manipulation of env vars.
@@ -4117,6 +4120,48 @@ next_item:
}
}
+/* Forward certain options to offloading compilation. */
+
+static void
+forward_offload_option (size_t opt_index, const char *arg, bool validated)
+{
+ switch (opt_index)
+ {
+ case OPT_l:
+ /* Use a '_GCC_' prefix and standard name ('-l_GCC_m' irrespective of the
+ host's 'MATH_LIBRARY', for example), so that the 'mkoffload's can tell
+ this has been synthesized here, and translate/drop as necessary. */
+ /* Note that certain libraries ('-lc', '-lgcc', '-lgomp', for example)
+ are injected by default in offloading compilation, and therefore not
+ forwarded here. */
+ /* GCC libraries. */
+ if (/* '-lgfortran' */ strcmp (arg, "gfortran") == 0 )
+ save_switch (concat ("-foffload-options=-l_GCC_", arg, NULL),
+ 0, NULL, validated, true);
+ /* Other libraries. */
+ else
+ {
+ /* The case will need special consideration where on the host
+ '!need_math', but for offloading compilation still need
+ '-foffload-options=-l_GCC_m'. The problem is that we don't get
+ here anything like '-lm', because it's not synthesized in
+ 'gcc/fortran/gfortranspec.cc:lang_specific_driver', for example.
+ Generally synthesizing '-foffload-options=-l_GCC_m' etc. in the
+ language specific drivers is non-trivial, needs very careful
+ review of their options handling. However, this issue is not
+ actually relevant for the current set of supported host/offloading
+ configurations. */
+ int need_math = (MATH_LIBRARY[0] != '\0');
+ if (/* '-lm' */ (need_math && strcmp (arg, MATH_LIBRARY) == 0))
+ save_switch ("-foffload-options=-l_GCC_m",
+ 0, NULL, validated, true);
+ }
+ break;
+ default:
+ gcc_unreachable ();
+ }
+}
+
/* Handle a driver option; arguments and return value as for
handle_option. */
@@ -4375,6 +4420,17 @@ driver_handle_option (struct gcc_options *opts,
/* POSIX allows separation of -l and the lib arg; canonicalize
by concatenating -l with its arg */
add_infile (concat ("-l", arg, NULL), "*");
+
+ /* Forward to offloading compilation '-l[...]' flags for standard,
+ well-known libraries. */
+ /* Doing this processing here means that we don't get to see libraries
+ injected via specs, such as '-lquadmath' injected via
+ '[build]/[target]/libgfortran/libgfortran.spec'. However, this issue
+ is not actually relevant for the current set of host/offloading
+ configurations. */
+ if (ENABLE_OFFLOADING)
+ forward_offload_option (opt_index, arg, validated);
+
do_save = false;
break;
diff --git a/libgomp/testsuite/libgomp.c/simd-math-1.c b/libgomp/testsuite/libgomp.c/simd-math-1.c
index 947bf606e36..dd2077cc597 100644
--- a/libgomp/testsuite/libgomp.c/simd-math-1.c
+++ b/libgomp/testsuite/libgomp.c/simd-math-1.c
@@ -4,7 +4,6 @@
/* { dg-do run } */
/* { dg-options "-O2 -ftree-vectorize -fno-math-errno" } */
/* { dg-additional-options -foffload-options=amdgcn-amdhsa=-mstack-size=3000000 { target offload_target_amdgcn } } */
-/* { dg-additional-options -foffload-options=-lm } */
#undef PRINT_RESULT
#define VERBOSE 0
diff --git a/libgomp/testsuite/libgomp.fortran/fortran-torture_execute_math.f90 b/libgomp/testsuite/libgomp.fortran/fortran-torture_execute_math.f90
index 3348a0bb3ad..2d0caa6b84a 100644
--- a/libgomp/testsuite/libgomp.fortran/fortran-torture_execute_math.f90
+++ b/libgomp/testsuite/libgomp.fortran/fortran-torture_execute_math.f90
@@ -1,4 +1,3 @@
! { dg-do run }
-! { dg-additional-options -foffload-options=-lm }
include '../../../gcc/testsuite/gfortran.fortran-torture/execute/math.f90'
diff --git a/libgomp/testsuite/libgomp.fortran/fortran.exp b/libgomp/testsuite/libgomp.fortran/fortran.exp
index e69656bce23..32e4bb2af4e 100644
--- a/libgomp/testsuite/libgomp.fortran/fortran.exp
+++ b/libgomp/testsuite/libgomp.fortran/fortran.exp
@@ -28,7 +28,6 @@ if { $blddir != "" } {
set lang_source_re {^.*\.[fF](|90|95|03|08)$}
set lang_include_flags "-fintrinsic-modules-path=${blddir}"
}
-set lang_link_flags "-foffload=-lgfortran"
lappend ALWAYS_CFLAGS "compiler=$GFORTRAN_UNDER_TEST"
# Initialize dg.
@@ -63,7 +62,6 @@ if { $blddir != "" } {
unset libquadmath_library_path
unset lang_library_paths
}
-unset lang_link_flags
# All done.
dg-finish
diff --git a/libgomp/testsuite/libgomp.oacc-fortran/fortran-torture_execute_math.f90 b/libgomp/testsuite/libgomp.oacc-fortran/fortran-torture_execute_math.f90
index 1b2ac440762..edc940c4706 100644
--- a/libgomp/testsuite/libgomp.oacc-fortran/fortran-torture_execute_math.f90
+++ b/libgomp/testsuite/libgomp.oacc-fortran/fortran-torture_execute_math.f90
@@ -1,5 +1,4 @@
! { dg-do run }
!TODO { dg-prune-output {using 'vector_length \(32\)', ignoring 1} }
-! { dg-additional-options -foffload-options=-lm }
include '../../../gcc/testsuite/gfortran.fortran-torture/execute/math.f90'
diff --git a/libgomp/testsuite/libgomp.oacc-fortran/fortran.exp b/libgomp/testsuite/libgomp.oacc-fortran/fortran.exp
index 2e8aa2285fb..e5844ad00fa 100644
--- a/libgomp/testsuite/libgomp.oacc-fortran/fortran.exp
+++ b/libgomp/testsuite/libgomp.oacc-fortran/fortran.exp
@@ -30,7 +30,6 @@ if { $blddir != "" } {
set lang_source_re {^.*\.[fF](|90|95|03|08)$}
set lang_include_flags "-fintrinsic-modules-path=${blddir}"
}
-set lang_link_flags "-foffload=-lgfortran"
lappend ALWAYS_CFLAGS "compiler=$GFORTRAN_UNDER_TEST"
# Initialize dg.
@@ -113,7 +112,6 @@ if { $blddir != "" } {
unset libquadmath_library_path
unset lang_library_paths
}
-unset lang_link_flags
# All done.
dg-finish
--
2.34.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [ping] driver: Forward '-lgfortran', '-lm' to offloading compilation
2023-06-13 10:44 ` [ping] " Thomas Schwinge
@ 2023-06-13 15:34 ` Joseph Myers
2023-06-14 10:13 ` Tobias Burnus
1 sibling, 0 replies; 4+ messages in thread
From: Joseph Myers @ 2023-06-13 15:34 UTC (permalink / raw)
To: Thomas Schwinge; +Cc: gcc-patches, Jakub Jelinek, Tobias Burnus, fortran
On Tue, 13 Jun 2023, Thomas Schwinge wrote:
> Hi!
>
> On 2023-06-05T14:25:18+0200, I wrote:
> > OK to push the attached
> > "driver: Forward '-lgfortran', '-lm' to offloading compilation"?
> > (We didn't have a PR open for that, or did we?)
>
> Ping.
OK.
--
Joseph S. Myers
joseph@codesourcery.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [ping] driver: Forward '-lgfortran', '-lm' to offloading compilation
2023-06-13 10:44 ` [ping] " Thomas Schwinge
2023-06-13 15:34 ` Joseph Myers
@ 2023-06-14 10:13 ` Tobias Burnus
1 sibling, 0 replies; 4+ messages in thread
From: Tobias Burnus @ 2023-06-14 10:13 UTC (permalink / raw)
To: Thomas Schwinge, gcc-patches, Joseph Myers, Jakub Jelinek
On 13.06.23 12:44, Thomas Schwinge wrote:
> On 2023-06-05T14:25:18+0200, I wrote:
>> OK to push the attached
>> "driver: Forward '-lgfortran', '-lm' to offloading compilation"?
>> (We didn't have a PR open for that, or did we?)
(It was approved by Joseph and pushed by Thomas as r14-1807-g4bcb46b3ade179 )
I wonder whether we should do for the example:
--- a/gcc/doc/invoke.texi
+++ b/gcc/doc/invoke.texi
@@ -2720 +2720 @@ Typical command lines are
--foffload-options=amdgcn-amdhsa=-march=gfx906 -foffload-options=-O3
+-foffload-options=amdgcn-amdhsa=-march=gfx906
To my knowledge the merge_flto_options is also run for code only doing
offloading - such that a host-side -O2 still ends up as -O2 for the offloading
compiler.
Thus, adding -foffload-options=-O3 encourages bad practice, at least kind of.
Thoughts?
BTW: I think the changed linking behavior should be document in the release
notes and in the wiki, i.e. https://gcc.gnu.org/gcc-14/changes.html
and https://gcc.gnu.org/wiki/Offloading (anywhere else?)
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
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-06-14 10:13 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-06-05 12:25 driver: Forward '-lgfortran', '-lm' to offloading compilation Thomas Schwinge
2023-06-13 10:44 ` [ping] " Thomas Schwinge
2023-06-13 15:34 ` Joseph Myers
2023-06-14 10:13 ` Tobias Burnus
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).