[Openmp-commits] [PATCH] D94573: [OpenMP] Drop the static library libomptarget-nvptx

Tue Jan 12 19:59:36 PST 2021

JonChesterfield accepted this revision.
JonChesterfield added a comment.
This revision is now accepted and ready to land.

@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++).

================
Comment at: openmp/libomptarget/deviceRTLs/nvptx/CMakeLists.txt:115
     "Enable CUDA LLVM bitcode offloading device RTL.")
   if (${LIBOMPTARGET_NVPTX_ENABLE_BCLIB})
     if (NOT ${LIBOMPTARGET_NVPTX_BCLIB_SUPPORTED})
----------------
We should probably rework this logic / failure reporting. Finding a working compiler should be simpler if we build libomptarget via ENABLE_RUNTIMES.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94573/new/

https://reviews.llvm.org/D94573