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

[Paul Robinson via Phabricator via llvm-commits]

probinson added inline comments.


================
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.
----------------
jdenny wrote:
> 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.
They can be evaluated in either order; but if they are both true, which one wins?  The one that has precedence.


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

https://reviews.llvm.org/D107162