[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