disabling lib/libomptarget.dylib build?

Currently trunk fails to build openmp on darwin due to the failure of...

[ 46%] Linking CXX shared library ../../../lib/libomptarget.dylib
cd /sw/src/fink.build/llvm50-5.0.0-1/build/stage1/projects/openmp/libomptarget
&& /sw/bin/cmake -E cmake_link_script
CMakeFiles/omptarget.dir/link.txt --verbose=1
/sw/src/fink.build/llvm50-5.0.0-1/opt-bin/ccclang++ -fno-common -fPIC
-fvisibility-inlines-hidden -Wall -W -Wno-unused-parameter
-Wwrite-strings -Wcast-qual -Wmissing-field-initializers -pedantic
-Wno-long-long -Wcovered-switch-default -Wnon-virtual-dtor
-Wdelete-non-virtual-dtor -Wstring-conversion -Werror=date-time
-std=c++11 -O3 -dynamiclib -Wl,-headerpad_max_install_names -L/sw/lib
-o ../../../lib/libomptarget.dylib -install_name
/sw/src/fink.build/llvm50-5.0.0-1/build/stage1/lib/libomptarget.dylib
CMakeFiles/omptarget.dir/src/omptarget.cpp.o -ldl
-Wl,--version-script=/sw/src/fink.build/llvm50-5.0.0-1/llvm-5.0.0.src/projects/openmp/libomptarget/exports
ld: unknown option:
--version-script=/sw/src/fink.build/llvm50-5.0.0-1/llvm-5.0.0.src/projects/openmp/libomptarget/exports
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make[2]: *** [lib/libomptarget.dylib] Error 1
make[1]: *** [projects/openmp/libomptarget/CMakeFiles/omptarget.dir/all] Error 2
make: *** [all] Error 2

when the cmake build attempts to use the unsupported -version-script
linker option on darwin. Is there a cmake variable which can be set to
suppress the build of libomptarget.dylib until this issue is fixed
properly?
         Jack
ps This issue is filed as http://bugs.llvm.org/show_bug.cgi?id=32309

Bug 32309 - ld: unknown option: --version-script= failure in
lib/libomptarget.dylib on darwin

Hi Jack,

*Is there a cmake variable which can be set to suppress the build of libomptarget.dylib until this issue is fixed properly?*

Yes, it is easy to fix. I’ll submit a patch.

George

graycol.gifJack Howarth via Openmp-dev —16/03/2017 09:18:41—Currently trunk fails to build openmp on darwin due to the failure of… [ 46%] Linking CXX shared l

Are you planning to retain the building of libomptarget.dylib on darwin by
avoiding the use of the unsupported -version-script linker option?
      Jack
ps Currently I am working around this issue by using the change...

Index: CMakeLists.txt

graycol.gif

I have already submitted a patch here: https://reviews.llvm.org/D31055

Which does exactly what you propose. My rationale is that, since the device plugins (CUDA, generic-elf-64) require an ELF-based system, building libomptarget on Windows or MacOS serves no purpose at this point - even it the library is built you cannot use it due to the lack of supported plugins. If someone in the future develops a Windows or MacOS plugin then we can revise this scheme.

George

graycol.gifJack Howarth —17/03/2017 14:49:59—On Thu, Mar 16, 2017 at 1:36 PM, Georgios Rokos grokos@us.ibm.com wrote: > Hi Jack,