Adding the config initialization (the footer) to clang-tools-extra so that tests that use REQUIRES, UNSUPPORTED, and XFAIL based on platform or target triple work properly.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
Nice catch!
I wonder how many more test/lit.site.cfg.in are silently partially non-functional because they don't have that substitution...
I also have an idea why it might not have worked for you - are you building clang-tools-extra repo separately manually, not as part of the whole llvm build?
Comment Actions
As per https://github.com/llvm-project/llvm-project-20170507/search?utf8=%E2%9C%93&q=LIT_SITE_CFG_IN_FOOTER&type=
this substitution already exists in most of the repos.
I don't see why it should not be in *all* top-level test/lit.site.cfg.in's.
(if it should not be, why is it in clang's, even though llvm's already has it?)
Though, here (c-t-e is built as part of the llvm), REQUIRES: windows works as expected.