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

[Kamil Rytarowski via Phabricator via llvm-commits]

krytarowski added a comment.

We attempted to patch internally LLD rebasing our code for the buildbot purposes but LLD without being used rots quickly, see: https://reviews.llvm.org/D58892

We use LD as `ld` and there are no plans neither intentions to break it. It was already decided internally in the project.

If you insist, I can rewrite nb.lld from C++ to shell and upstream to the LLD repository. We require `-fuse-ld=lld` to be rerouted through this shell script and lld invocation call it so we can revert back to the concept from https://reviews.llvm.org/D69755. Personally I see no gain in this as I prefer to rely on LLVM libs at least for Target Triplet detection.

As I can understand that Linux/GNU behavior is promoted (and enforced to other ELF OSs) it does not work for us. I propose a patch that keeps intact the GNU/Linux code.

If 200 LOC for NetBSD is a show stopper, please see e.g. compiler-rt where we maintain 2 orders of magnitude more NetBSD-only LOC.

I try to avoid forking LLD for downstream and this shall be appreciated.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70048