aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorThomas Schwinge <thomas@codesourcery.com>2021-03-03 22:37:58 +0100
committerThomas Schwinge <thomas@codesourcery.com>2021-03-25 12:06:42 +0100
commitff77b4a0db75bc82a5519e31a882f9a25a02cd56 (patch)
tree6205af8efd7fe76d780a5d7df3420069bf86498e
parentb0e5c3b84ef2c477fe797da59a1aadfbed8445fe (diff)
downloadgcc-ff77b4a0db75bc82a5519e31a882f9a25a02cd56.zip
gcc-ff77b4a0db75bc82a5519e31a882f9a25a02cd56.tar.gz
gcc-ff77b4a0db75bc82a5519e31a882f9a25a02cd56.tar.bz2
[WIP] OpenMP 5.0: requires directive: workaround to fix libgomp IntelMIC plugin build
Fix-up for og10 commit c2e4a17adc0989f216c7fc3f93f150c66adba23a "OpenMP 5.0: requires directive". The GCC offloading target configurations don't build/use 'crtoffloadbegin.o'/'crtoffloadtable.o'/'crtoffloadend.o' ('libgcc/offloadstuff.c'), but the libgomp IntelMIC plugin still does link against libgomp, and the latter unconditionally refers to '__requires_mask_table', '__requires_mask_table_end': make[3]: Entering directory '[...]/build-gcc-offload-x86_64-intelmicemul-linux-gnu/x86_64-intelmicemul-linux-gnu/liboffloadmic/plugin' [...]/build-gcc-offload-x86_64-intelmicemul-linux-gnu/./gcc/xg++ [...] -loffloadmic_target -lcoi_device -lgomp -rdynamic ../ofldbegin.o offload_target_main.o ../ofldend.o -o offload_target_main ./../../libgomp/.libs/libgomp.so: undefined reference to `__requires_mask_table_end' ./../../libgomp/.libs/libgomp.so: undefined reference to `__requires_mask_table' collect2: error: ld returned 1 exit status Makefile:806: recipe for target 'offload_target_main' failed make[3]: *** [offload_target_main] Error 1 I have not researched what a proper fix would look like. libgomp/ * target.c (__requires_mask_table, __requires_mask_table_end): Add '__attribute__((weak))'.
-rw-r--r--libgomp/ChangeLog.omp5
-rw-r--r--libgomp/target.c2
2 files changed, 7 insertions, 0 deletions
diff --git a/libgomp/ChangeLog.omp b/libgomp/ChangeLog.omp
index 0e3fd12..03ca74c 100644
--- a/libgomp/ChangeLog.omp
+++ b/libgomp/ChangeLog.omp
@@ -1,3 +1,8 @@
+2021-03-25 Thomas Schwinge <thomas@codesourcery.com>
+
+ * target.c (__requires_mask_table, __requires_mask_table_end): Add
+ '__attribute__((weak))'.
+
2021-03-01 Kwok Cheung Yeung <kcy@codesourcery.com>
* testsuite/libgomp.c-c++-common/collapse-4.c: New.
diff --git a/libgomp/target.c b/libgomp/target.c
index 699dc72..9c75826 100644
--- a/libgomp/target.c
+++ b/libgomp/target.c
@@ -104,7 +104,9 @@ static unsigned int gomp_requires_mask;
/* Start/end of .gnu.gomp.requires section of program, defined in
crtoffloadbegin/end.o. */
+__attribute__((weak))
extern const unsigned int __requires_mask_table[];
+__attribute__((weak))
extern const unsigned int __requires_mask_table_end[];
/* Similar to gomp_realloc, but release register_lock before gomp_fatal. */