[PATCH] D71408: [lit] Remove lit's REQUIRES-ANY directive
Greg Parker via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Dec 16 22:00:05 PST 2019
gparker42 accepted this revision.
gparker42 added a comment.
This revision is now accepted and ready to land.
Good. `REQUIRES-ANY` was preserved when the boolean expressions were added because libc++ was still using it. libc++ has since changed their tests so removing it should be fine now.
(previous discussion: D18185 <https://reviews.llvm.org/D18185>)
The risk for any remaining users of `REQUIRES-ANY` is that they will see a test run too often. That's an acceptable failure mode (unlike silently not running the test at all).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71408/new/
https://reviews.llvm.org/D71408
More information about the cfe-commits
mailing list