This is an archive of the discontinued LLVM Phabricator instance.

[lld] respect LLVM_EXTERNAL_LIT
ClosedPublic

Authored by kwk on Jul 6 2023, 5:12 AM.

Details

Summary

Consider a setup without a system-wide installation of lit.
Instead you pass the path to lit like this:

cmake ...
-DLLVM_EXTERNAL_LIT=<PATH_TO_LIT_BINARY> ...

Then you will run into this error:

ninja: error: unknown target 'check-lld'

I have a buildbot builder that fails with this message. Here's the
passage that triggers this error:

https://github.com/llvm/llvm-zorg/blob/d3bfd5ccbceb542098c350e4d071ceceac6854cb/zorg/buildbot/builders/annotated/standalone-build.sh#L194-L239

By using LLVM_EXTERNAL_LIT instead of LLVM_LIT we fix this problem.

See
here for
a description:

Both the LLVM_ROOT and LLVM_EXTERNAL_LIT options are required to do stand-alone builds for all sub-projects. Additional required options for each sub-project can be found in the table below.

Diff Detail

Event Timeline

kwk created this revision.Jul 6 2023, 5:12 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 6 2023, 5:12 AM
kwk requested review of this revision.Jul 6 2023, 5:12 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 6 2023, 5:12 AM
tstellar accepted this revision.Jul 6 2023, 6:37 AM

LGTM.

This revision is now accepted and ready to land.Jul 6 2023, 6:37 AM
This revision was automatically updated to reflect the committed changes.