[Openmp-commits] [PATCH] D95466: [OpenMP][NVPTX] Drop dependence on CUDA to build NVPTX `deviceRTLs`

[Jon Chesterfield via Phabricator via Openmp-commits]

JonChesterfield added a comment.

In D95466#2525874 <https://reviews.llvm.org/D95466#2525874>, @kparzysz wrote:

> I don't have CUDA, why is this being compiled on my system to begin with?

The idea is to compile llvm systems that don't have cuda installed, such that the toolchain can later compile openmp code that runs on systems that do have cuda + nvptx hardware.

In particular, so that the llvm compiled for linux distributions can be installed from a package manager onto a system that does have a gpu.

However, it's starting to look like some people who are building libomptarget don't have gcc-multilibs installed and also don't care about nvptx offloading.

I think we therefore need to do one of the following:

- turn this build off by default if cuda is missing (what we used to have)
- turn this build off if compilation fails, e.g. by trying to detect multilibs, or otherwise make compilation non-fatal
- drop the dependency on the host, which is straightforward if we disable printf and otherwise awkward


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95466