krytarowski added a comment. In D56215#1345326 <https://reviews.llvm.org/D56215#1345326>, @ruiu wrote:
> In D56215#1344279 <https://reviews.llvm.org/D56215#1344279>, @krytarowski > wrote: > > > In D56215#1344233 <https://reviews.llvm.org/D56215#1344233>, @ruiu wrote: > > > > > lld's driver is intentionally designed to be agnostic of the host that > > > the linker is running for the reasons described at the beginning of > > > Driver.cpp: > > > https://github.com/llvm-project/lld/blob/master/ELF/Driver.cpp#L13 I > > > think I like that approach. If you need to do this, you can do this in > > > the compiler driver rather than in the linker itself. Is there any reason > > > you need to do this in the linker? > > > > > > This breaks compat with GNU ld here and the linker is intended to be used > > standalone. > > > This is where lld is not 100% compatible with GNU ld, but I'd think that's > not a bad thing. I'd like to make lld agnostics of host OS so that the linker > works exactly in the same way on any operating systems, which makes cross > linking much easier to do. So, both a run-time detection of a host OS or a > configure-time customization are I think undesirable. Personally I have no strong opinion either way. Joerg is blocking a patch to handle it in clang: https://reviews.llvm.org/D33726 I find it very silly reason to brick NetBSD support in lld. I expect that it's easier to push paths into lld than convince @joerg to accept clang patch. Repository: rLLD LLVM Linker CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56215/new/ https://reviews.llvm.org/D56215 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
