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

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 13 16:57:11 PDT 2020


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





More information about the llvm-commits mailing list