[PATCH] D79328: lld: Let find_package(LLD) work

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added a comment.

I usually just follow the executable name and call it `lld`. In patch subjects, I follow pre-git layout and just use `[ELF] ` and don't bother adding `[lld] `.
I see `LLD` is common, too. `LLD` is the string `lld-link` and `ld.lld` --version print:

% ld.lld --version
LLD 11.0.0 (compatible with GNU linkers)

I think either is ok.

--------

For this patch, what does it intend to achieve? `find_package(LLD)`, link against `lldCOFF`, `lldELF` etc, and call `lld::elf::link`, `lld::wasm::link`?

It'd be nice if you have an out-of-tree project or something which demonstrates that `find_package(LLD)` works, if you do want to use lld this way.


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

https://reviews.llvm.org/D79328