krytarowski added a comment.

I don't want to diverge this patch on offtopic or general discussion.

Does it look good for landing? We need this support to be functional out of the 
box and as a part of LLVM. E.g. `clang -fuse-ld=` currently tries to call 
`ld.lld`, with this patch it will go through `nb.lld` as expected.

`--no-rosegment` is fixed with our WIP patches, but we NEED any support in LLD 
as a starting point so we can later just detach it (or add a check for version 
of NetBSD as we need Triple). Later we will the set the options on our own. I 
actually forgot to e.g. disable GNU stack and keep delaying it until the patch 
will land the sources.

We e.g. miss certain `-switches` in LLD, but hard to do anything in LLD without 
starting point. They will be proposed in future.


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