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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
