[PATCH] D107162: [lit] Have REQUIRES support the target triple

[Joel E. Denny via Phabricator via llvm-commits]

jdenny added a comment.





================
Comment at: llvm/docs/TestingGuide.rst:472-473
 | ``REQUIRES`` enables the test if all expressions are true.
-| ``UNSUPPORTED`` disables the test if any expression is true.
+| ``UNSUPPORTED`` disables the test if any expression is true, and takes
+precedence over ``REQUIRES``.
 | ``XFAIL`` expects the test to fail if any expression is true.
----------------
I'm not sure this comment about precedence is meaningful.  It seems to say that, if you have an UNSUPPORTED directive, any REQUIRES directive is ignored.  Actually, in the implementation, REQUIRES is checked first.

I think there's no sense of precedence.  The test runs if REQUIRES is matched and UNSUPPORTED isn't matched.  I think they could be evaluated in either order.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107162/new/

https://reviews.llvm.org/D107162