[OpenMP] Drop the static library libomptarget-nvptx

[tianshilei1992]

Summary

For NVPTX target, OpenMP provides a static library libomptarget-nvptx
built by NVCC, and another bitcode libomptarget-nvptx-sm_{$sm}.bc generated by
Clang. When compiling an OpenMP program, the .bc file will be fed to clang
in the second run on the program that compiles the target part. Then the generated
PTX file will be fed to ptxas to generate the object file, and finally the driver
invokes nvlink to generate the binary, where the static library will be appened
to nvlink.

One question is, why do we need two libraries? The only difference is, the static
library contains omp_data.cu and the bitcode library doesn't. It's unclear why
they were implemented in this way, but per D94565, there is no issue if we also
include the file into the bitcode library. Therefore, we can safely drop the
static library.

This patch is about the change in OpenMP. The driver will be updated as well if
this patch is accepted.

[grokos]

One question is, why do we need two libraries? The only difference is, the static library contains omp_data.cu and the bitcode library doesn't. It's unclear why they were implemented in this way

It's been a very long time since then (6+ years), but if memory serves me well the reason we needed to build the static library in the early days was that a lot of systems lacked a CUDA compiler capable of emitting LLVM bitcode, so the static library was the only choice. Things have changed a lot in the meantime, so I guess the static library fallback is irrelevant nowadays.

[JonChesterfield]

@grokos that sounds right. As far as I know, clang is capable of compiling this correctly as cuda (otherwise we'd have bug reports from people using the bitcode build).

Dropping this is a step towards compiling the deviceRTL as openmp (or c++).

openmp/libomptarget/deviceRTLs/nvptx/CMakeLists.txt

115

We should probably rework this logic / failure reporting. Finding a working compiler should be simpler if we build libomptarget via ENABLE_RUNTIMES.

[grokos]

Also, since D94565 is merged now, this patch needs rebasing :)

[jdoerfert]

Nice

[JonChesterfield]

Discussed on weekly call, we have consensus to ship this.

[tianshilei1992]

Rebased the patch and removed unnecessary part

[tianshilei1992]

Optimized some logics

openmp/libomptarget/deviceRTLs/nvptx/CMakeLists.txt

if(LIBOMPTARGET_DEP_CUDA_FOUND)
   set(CUDA_PROPAGATE_HOST_FLAGS OFF)
 
   set(cuda_src_files
       ${devicertl_common_directory}/src/cancel.cu
       ${devicertl_common_directory}/src/critical.cu
       ${devicertl_common_directory}/src/data_sharing.cu
       ${devicertl_common_directory}/src/libcall.cu
       ${devicertl_common_directory}/src/loop.cu
+      ${devicertl_common_directory}/src/omp_data.cu
       ${devicertl_common_directory}/src/omptarget.cu
       ${devicertl_common_directory}/src/parallel.cu
       ${devicertl_common_directory}/src/reduction.cu
       ${devicertl_common_directory}/src/support.cu
       ${devicertl_common_directory}/src/sync.cu
       ${devicertl_common_directory}/src/task.cu
       src/target_impl.cu
   )
 
-  set(omp_data_objects ${devicertl_common_directory}/src/omp_data.cu)
-
   # Build library support for the highest compute capability the system supports
   # and always build support for sm_35 by default
   if (${LIBOMPTARGET_DEP_CUDA_ARCH} EQUAL 35)
     set(default_capabilities 35)
   else()
       set(default_capabilities "35,${LIBOMPTARGET_DEP_CUDA_ARCH}")
   endif()
 
if(LIBOMPTARGET_DEP_CUDA_FOUND)
   # Override default MAX_SM in src/target_impl.h if requested
   if (DEFINED LIBOMPTARGET_NVPTX_MAX_SM)
     set(MAX_SM_DEFINITION "-DMAX_SM=${LIBOMPTARGET_NVPTX_MAX_SM}")
   endif()
 
   # Activate RTL message dumps if requested by the user.
   set(LIBOMPTARGET_NVPTX_DEBUG FALSE CACHE BOOL
     "Activate NVPTX device RTL debug messages.")
-  if(${LIBOMPTARGET_NVPTX_DEBUG})
-    set(CUDA_DEBUG -DOMPTARGET_NVPTX_DEBUG=-1 -g --ptxas-options=-v)
-  endif()
-
-  # NVPTX runtime library has to be statically linked. Dynamic linking is not
-  # yet supported by the CUDA toolchain on the device.
-  set(BUILD_SHARED_LIBS OFF)
-  set(CUDA_SEPARABLE_COMPILATION ON)
-  list(APPEND CUDA_NVCC_FLAGS -I${devicertl_base_directory}
-                              -I${devicertl_nvptx_directory}/src)
-  cuda_add_library(omptarget-nvptx STATIC ${cuda_src_files} ${omp_data_objects}
-      OPTIONS ${CUDA_ARCH} ${CUDA_DEBUG} ${MAX_SM_DEFINITION})
-
-  # Install device RTL under the lib destination folder.
-  install(TARGETS omptarget-nvptx ARCHIVE DESTINATION "${OPENMP_INSTALL_LIBDIR}")
-
-  target_link_libraries(omptarget-nvptx ${CUDA_LIBRARIES})
-
 
   # Check if we can create an LLVM bitcode implementation of the runtime library
   # that could be inlined in the user application. For that we need to find
   # a Clang compiler capable of compiling our CUDA files to LLVM bitcode and
   # an LLVM linker.
   set(LIBOMPTARGET_NVPTX_CUDA_COMPILER "" CACHE STRING
     "Location of a CUDA compiler capable of emitting LLVM bitcode.")
   set(LIBOMPTARGET_NVPTX_BC_LINKER "" CACHE STRING
     "Location of a linker capable of linking LLVM bitcode objects.")
 
   include(LibomptargetNVPTXBitcodeLibrary)
 
   set(bclib_default FALSE)
   if (${LIBOMPTARGET_NVPTX_BCLIB_SUPPORTED})
     set(bclib_default TRUE)
   endif()
   set(LIBOMPTARGET_NVPTX_ENABLE_BCLIB ${bclib_default} CACHE BOOL
     "Enable CUDA LLVM bitcode offloading device RTL.")
   if (${LIBOMPTARGET_NVPTX_ENABLE_BCLIB})
     if (NOT ${LIBOMPTARGET_NVPTX_BCLIB_SUPPORTED})
       libomptarget_error_say("Cannot build CUDA LLVM bitcode offloading device RTL!")
     endif()
     libomptarget_say("Building CUDA LLVM bitcode offloading device RTL.")
 
     # Set flags for LLVM Bitcode compilation.
     set(bc_flags ${LIBOMPTARGET_NVPTX_SELECTED_CUDA_COMPILER_FLAGS}
                  -I${devicertl_base_directory}
foreach(sm ${nvptx_sm_list})
       set_property(DIRECTORY APPEND PROPERTY ADDITIONAL_MAKE_CLEAN_FILES libomptarget-nvptx-sm_${sm}.bc)
 
       add_custom_target(omptarget-nvptx-${sm}-bc ALL DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/libomptarget-nvptx-sm_${sm}.bc)
       add_dependencies(omptarget-nvptx-bc omptarget-nvptx-${sm}-bc)
 
       # Copy library to destination.
       add_custom_command(TARGET omptarget-nvptx-${sm}-bc POST_BUILD
                          COMMAND ${CMAKE_COMMAND} -E copy ${CMAKE_CURRENT_BINARY_DIR}/libomptarget-nvptx-sm_${sm}.bc
-                         $<TARGET_FILE_DIR:omptarget-nvptx>)
+                         ${LIBOMPTARGET_LIBRARY_DIR})
 
       # Install bitcode library under the lib destination folder.
       install(FILES ${CMAKE_CURRENT_BINARY_DIR}/libomptarget-nvptx-sm_${sm}.bc DESTINATION "${OPENMP_INSTALL_LIBDIR}")
     endforeach()
   endif()
 
   add_subdirectory(test)
 else()
   libomptarget_say("Not building CUDA offloading device RTL: CUDA tools not found in the system.")
 endif()