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

Repository
rL LLVM

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.