This is needed when the external projects try to use other tools
besides just the compiler and the linker.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
LGTM
llvm/cmake/modules/LLVMExternalProjectUtils.cmake | ||
---|---|---|
111 ↗ | (On Diff #150157) | Do we only care about ELF? |
llvm/cmake/modules/LLVMExternalProjectUtils.cmake | ||
---|---|---|
111 ↗ | (On Diff #150157) | Ideally this would support other platforms as well, but there are other bits in this function that would need to be probably changed as well, e.g. CMAKE_C_COMPILER and CMAKE_CXX_COMPILER should be also pointing to clang-cl when target system is Windows, so I'm not sure if it's worth doing all that in this patch? |
llvm/cmake/modules/LLVMExternalProjectUtils.cmake | ||
---|---|---|
111 ↗ | (On Diff #150157) | Yeah, that's fair. This is fine as-is; I was just wondering in general. |