The patch in D152592 changed the logic for this. We could never check if
we were on the GPU as this was before the variable was defined so I
moved it later. Secondly, we cannot use the LLVM_BINARY_DIR here, and
I do not know if that works in general. The problem is that it will
isntall the headers under a normal path outside of the
LLVM_ENABLE_RUNTIMES build. I don't know if that's correct for the
other targets, but for the GPU I need to set it back to the
CMAKE_BINARY_DIR so it works.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
libc/CMakeLists.txt | ||
---|---|---|
90 | Should this also be changed to ${CMAKE_BINARY_DIR}/lib? I'd expect for either both LLVM_BINARY_DIR and LLVM_LIBRARY_OUTPUT_INTDIR to be defined, or neither. |
libc/CMakeLists.txt | ||
---|---|---|
90 | I think previously it was the one with the host triple, but it should probably just go under /lib since we call it libcgpu.a. |
Should this also be changed to ${CMAKE_BINARY_DIR}/lib? I'd expect for either both LLVM_BINARY_DIR and LLVM_LIBRARY_OUTPUT_INTDIR to be defined, or neither.