[OpenMP] Dropped unnecessary define when compiling deviceRTLs for NVPTX
authorShilei Tian <tianshilei1992@gmail.com>
Thu, 14 Jan 2021 18:55:05 +0000 (13:55 -0500)
committerShilei Tian <tianshilei1992@gmail.com>
Thu, 14 Jan 2021 18:55:12 +0000 (13:55 -0500)
The comment said CUDA 9 header files use the `nv_weak` attribute which
`clang` is not yet prepared to handle. It's three years ago and now things have
changed. Based on my test, removing the definition doesn't have any problem on
my machine with CUDA 11.1 installed.

Reviewed By: jdoerfert

Differential Revision: https://reviews.llvm.org/D94700

openmp/libomptarget/deviceRTLs/nvptx/CMakeLists.txt

index 200c640..c8acf6a 100644 (file)
@@ -89,13 +89,6 @@ if(LIBOMPTARGET_DEP_CUDA_FOUND)
       set(bc_flags ${bc_flags} -DOMPTARGET_NVPTX_DEBUG=0)
     endif()
 
-    # CUDA 9 header files use the nv_weak attribute which clang is not yet prepared
-    # to handle. Therefore, we use 'weak' instead. We are compiling only for the
-    # device, so it should be equivalent.
-    if(CUDA_VERSION_MAJOR GREATER 8)
-      set(bc_flags ${bc_flags} -Dnv_weak=weak)
-    endif()
-
     # Create target to build all Bitcode libraries.
     add_custom_target(omptarget-nvptx-bc)