This string parameter is passed to -fuse-ld when linking. It can be
an absolute path to your custom linker, otherwise clang will look for
ld.{name}.
Details
Details
Diff Detail
Diff Detail
- Build Status
Buildable 2935 Build 2935: arc lint + arc unit
Event Timeline
Comment Actions
Restore unintended removal of warning on Apple when using LLD and LTO, and fix tailing whitespace.
Comment Actions
This is a nice improvement. Some comments inline.
llvm/cmake/modules/HandleLLVMOptions.cmake | ||
---|---|---|
99–101 | I think still emitting a diagnostic in this case would be good. In general, I think we should emit a diagnostic on every linker that can't do LTO when we try to do it. Why did you remove it? | |
llvm/docs/CMake.rst | ||
385–389 | Hmm, this can be expanded a bit. If clang is used as base compiler, what you say is perfectly fine.
|
Comment Actions
LGTM modulo nit.
llvm/cmake/modules/HandleLLVMOptions.cmake | ||
---|---|---|
156–158 | I think the correct spelling is doesn't |
I think still emitting a diagnostic in this case would be good. In general, I think we should emit a diagnostic on every linker that can't do LTO when we try to do it. Why did you remove it?