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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to