On Sat, Nov 09, 2019 at 01:07:24AM +0000, Maciej W. Rozycki wrote:
> --- gcc.orig/libgomp/testsuite/lib/libgomp.exp
> +++ gcc/libgomp/testsuite/lib/libgomp.exp
> @@ -174,6 +174,16 @@ proc libgomp_init { args } {
>      # For build-tree testing, also consider the library paths used for 
> builing.
>      # For installed testing, we assume all that to be provided in the 
> sysroot.
>      if { $blddir != "" } {
> +     # Offload options imply `-pthread', and that implies `-latomic'
> +     # on some targets, so wire in libatomic build directories.

-fopenmp is not an option I'd like to call Offload option, OpenMP is much
more than just offloading, and this isn't on some targets, but only one, riscv*.
So, I think it should be done only for
that target and talk about -fopenmp/-fopenacc options instead of Offload
options.

> +     set shlib_ext [get_shlib_extension]
> +     set atomic_library_path "${blddir}/../libatomic/.libs"
> +     if { [file exists "${atomic_library_path}/libatomic.a"]
> +          || [file exists \
> +              "${atomic_library_path}/libatomic.${shlib_ext}"] } {
> +         lappend ALWAYS_CFLAGS "additional_flags=-L${atomic_library_path}"
> +         append always_ld_library_path ":${atomic_library_path}"
> +     }
>       global cuda_driver_include
>       global cuda_driver_lib
>       if { $cuda_driver_include != "" } {

        Jakub

Reply via email to