This is an archive of the discontinued LLVM Phabricator instance.

Use installed llvm-lit.py instead of lit.py PR-51072
ClosedPublic

Authored by yaron.keren on Aug 15 2021, 6:37 AM.

Details

Summary

Three tests fail when building and testing LLVM from the Visual C++ environment
since they use the repo version of lit.py that do not have local customization
builtin_parameters = { 'build_mode' : 'Release' }
https://bugs.llvm.org/show_bug.cgi?id=51072

Diff Detail

Event Timeline

yaron.keren requested review of this revision.Aug 15 2021, 6:37 AM
yaron.keren created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptAug 15 2021, 6:37 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript

looking into Linux failures

Support both Windows llvm-lit.py and Linux llvm-lit command name

Switched to Linux repo

dyung accepted this revision.Aug 17 2021, 2:41 PM
dyung added a subscriber: dyung.

LGTM and I can confirm that this change fixes the check-globals.test that was failing due to this issue. Thanks!

This revision is now accepted and ready to land.Aug 17 2021, 2:41 PM
This revision was automatically updated to reflect the committed changes.
tstellar added inline comments.Aug 30 2021, 9:05 PM
clang/test/utils/update_cc_test_checks/lit.local.cfg
24

This patch breaks stand-alone clang builds, because it assumes llvm-lit* will be present in the build directory. This line should be guarded by if not config.llvm_external_lit.

yaron.keren marked an inline comment as done.Aug 31 2021, 8:04 AM
yaron.keren added inline comments.
clang/test/utils/update_cc_test_checks/lit.local.cfg
24