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