Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
CC Bob, I suspect we should add the support for this option for lld-link on windows as well.
llvm/cmake/modules/HandleLLVMOptions.cmake | ||
---|---|---|
724 ↗ | (On Diff #90269) | Is LLVM_USE_LINKER already normalized into lower case? It seems that STREQUAL is case sensitive. |
730 ↗ | (On Diff #90269) | I'm fine with this for now, but I feel we should provide a clang driver option to abstract away all this logic. |
llvm/cmake/modules/HandleLLVMOptions.cmake | ||
---|---|---|
724 ↗ | (On Diff #90269) | I don't think so, but I wouldn't normally expect something like -DLLVM_USE_LINKER=LLD to work anyway because this will translate into -fuse-ld=LLD which probably wouldn't work on case-sensitive file systems (which ELF targets normally are). |
730 ↗ | (On Diff #90269) | Agreed, I've added a FIXME. |