This is an archive of the discontinued LLVM Phabricator instance.

[lit] Allow specification of LLD variant paths via environment variable

Authored by jhenderson on May 6 2021, 6:50 AM.



With this patch, the environment variables LDLLD, LLDLINK, LD64LLD, and WASMLD can be used to specify the path to ld.lld, lld-link, ld64.lld and wasm-ld respectively, in a similar manner to how the CLANG variable points to clang.

Diff Detail

Event Timeline

jhenderson created this revision.May 6 2021, 6:50 AM
jhenderson requested review of this revision.May 6 2021, 6:50 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 6 2021, 6:50 AM
Herald added a subscriber: aheejin. · View Herald Transcript
thopre accepted this revision.May 6 2021, 8:36 AM
This revision is now accepted and ready to land.May 6 2021, 8:36 AM
thopre added inline comments.May 6 2021, 8:37 AM

This is already in the other patch, do you want to rebase this one on top?

jhenderson added inline comments.May 7 2021, 12:54 AM

Oops. Messed up making my diff. The actual version I've got locally is correct.

Fix diff. I'm going to hold off landing this for now until the discussion on the mailing list ( has reached a conclusion.

jhenderson abandoned this revision.May 17 2021, 2:05 AM

The discussion seems to be leaning towards not adding the variable unless necessary. As such, I think this patch can be abandoned.