[PATCH] D147581: [libc] Ensure that the required clang tools are up-to-date for libc GPU

[Artem Belevich via Phabricator via llvm-commits]

tra added inline comments.


================
Comment at: llvm/runtimes/CMakeLists.txt:403
+    if(LIBC_GPU_BUILD OR LIBC_GPU_ARCHITECTURES)
+      list(APPEND extra_deps clang-offload-packager nvptx-arch amdgpu-arch)
+    endif()
----------------
Do these tools depend on something that may not be present on a build machine w/o GPU?  

I think `nvptx-arch` currently links with libcuda.so, which means that it will not build on machine w/o a NVIDIA GPU.  Ideally the app needs to dlopen(libcuda.so.1) and gracefully fail if it's not found.


I think the dependency on nvptx-arch tools may need to be conditional on whether its build prerequisites are found.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147581