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

[Joseph Huber via Phabricator via libc-commits]

jhuber6 added inline comments.


================
Comment at: libc/utils/gpu/loader/nvptx/CMakeLists.txt:12-13
   CUDA::cuda_driver
+  LLVMObject
+  LLVMSupport
 )
----------------
tra wrote:
> 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?
When done in the `runtimes/runtimes-bins` directory.
```
[1/1] Cleaning all built files...
Cleaning... 509 files.
3
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149527