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

Sat Nov 2 15:49:22 PDT 2019

MaskRay added a comment.

I still have the feeling that such configurations should be added to clangDriver/gcc specs or a shell script wrapper of lld.

How do you decide to handle "Handling of indirect shared library dependencies"? It does not seem to be favored by lld contributors.

================
Comment at: lld/tools/nb.lld/nb.lld.cpp:94
+
+  // disable superfluous RUNPATH on NetBSD
+  args.push_back("--disable-new-dtags");
----------------
You can't use DT_RUNPATH probably because the change `Add support for DT_RUNPATH - it is just the same as we always have treated DT_RPATH.  Approved by core.` (2018-12-30) is still considered new. If you do this in clangDriver, you need a comment with a TODO about the migration process.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69755