[PATCH] D56215: [lld] [ELF] Include default search paths for NetBSD driver

Michał Górny via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 3 12:34:01 PST 2019


mgorny added a comment.

We've discussed this a bit and given other changes we need to do, and I see pretty much three options here:

1. We hardcode stuff under `defined(__NetBSD__)` which kinda solves the problem, except lld won't be very cross-friendly.
2. We try to do conditionals based on triple but this works only when we customize the install to include it in executable name. We probably would still need to default based on `defined(__NetBSD__)` when triple isn't available via process name.
3. We create `*nbsd*` emulations for all arches (e.g. amd64/x86 don't have such emulations right now), and use that to switch logic. This would be closer to what FreeBSD does, I think. However, if we do this, then I suppose we should also add similar aliases to GNU ld.


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

https://reviews.llvm.org/D56215





More information about the llvm-commits mailing list