This is an archive of the discontinued LLVM Phabricator instance.

[test] Adding config initialization to lit tests in clang-tools-extra
ClosedPublic

Authored by juliehockett on Mar 20 2018, 2:57 PM.

Details

Summary

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.

Diff Detail

Event Timeline

juliehockett created this revision.Mar 20 2018, 2:57 PM

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?

lebedev.ri accepted this revision.Mar 20 2018, 3:36 PM

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.

This revision is now accepted and ready to land.Mar 20 2018, 3:36 PM
This revision was automatically updated to reflect the committed changes.