[libc-commits] [PATCH] D149527: [libc] Support global constructors and destructors on NVPTX

[Artem Belevich via Phabricator via libc-commits]

tra added inline comments.


================
Comment at: libc/utils/gpu/loader/nvptx/CMakeLists.txt:12-13
   CUDA::cuda_driver
+  LLVMObject
+  LLVMSupport
 )
----------------
jhuber6 wrote:
> tra wrote:
> > Can you check how long the clean build of the tool with `-j 6` would take now? If it's in the ballpark of a minute or so, we can probably live with that. Otherwise we should build the tool along with clang/LLVM, similar to how we deal with `libc-hdrgen`.
> Is this assume we need to rebuild the libraries? I figured these would be copied somewhere in the build environment. On my machine rebuilding the loader takes about five seconds.
We only copy the clang installation directory to the GPU machines, so when you build the tests, the build directory itself will be empty and the libraries would have to be rebuilt.

> On my machine rebuilding the loader takes about five seconds.

Is that a clean build? Can you check what `ninja clean; ninja -nv nvptx_loader | wc -l` shows?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149527