[PATCH] D44708: [test] Adding config initialization to lit tests in clang-tools-extra

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 20 15:36:48 PDT 2018


lebedev.ri accepted this revision.
lebedev.ri added a comment.
This revision is now accepted and ready to land.

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.


https://reviews.llvm.org/D44708





More information about the cfe-commits mailing list