[PATCH] D70048: [LLD] Add NetBSD support as a new flavor of LLD (nb.lld)

Kamil Rytarowski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 13 17:16:33 PST 2019


krytarowski added a comment.

Stating it from a different side. If you do not care about GNU ld drop-in replacement property for Linux it's not our business, but we do care about this in NetBSD and we restrict this to our project only (from ELF targets).  If you do not care about NetBSD, you shall not care how and whether we use LLD. We try hard to not interfere with Linux looking for a consensus.

You are not forced to use the NetBSD driver neither define it in `LLD_SYMLINKS_TO_CREATE`. If you have a strong disrespect to this GNU ld drop-in replacement approach, I can patch the `die("")` invocation in `main()` to conditionally print `"nb.lld"`. There are around 20 lines patches in the generic code and around 1/3 of that is in the code comments. This shall not be burden for any of you.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70048/new/

https://reviews.llvm.org/D70048





More information about the llvm-commits mailing list