Allow conditional lit tests in Subzero, based on build flags.

tests_lit/llvm2ice_tests/fp.pnacl.ll

Index: tests_lit/llvm2ice_tests/fp.pnacl.ll
diff --git a/tests_lit/llvm2ice_tests/fp.pnacl.ll b/tests_lit/llvm2ice_tests/fp.pnacl.ll
index 474961a4b301e4d7d083bd3abbb687f3413141d8..ef40bae0b050fb5cb796dc20c0445fb8e8849426 100644
--- a/tests_lit/llvm2ice_tests/fp.pnacl.ll
+++ b/tests_lit/llvm2ice_tests/fp.pnacl.ll
@@ -7,6 +7,7 @@
 ; doesn't know how to symbolize non-section-local functions.
 ; The newer LLVM 3.6 one does work, but watch out for other bugs.
 
+; REQUIRES: x86-32

[jvoung (off chromium), 2014/10/15 21:57:06]

Any reason this is considered REQUIRES: x86-32 vs say, ebp_args.ll ?

This one seems like it could be extended to have more arches, while the ebp
sounds more specific to x86-32 (though could apply to anything with a frame
pointer, with some changes).

Which part of the CL enables the REQUIRES: syntax?

[Karl, 2014/10/16 16:51:48]

The point of this change was to show that requires works. Since it appears
confusing, I will remove it. However, the point is that in
Subzero/tests_lit/lit.cfg (line 74) updates config.available_features to include
all (target) build attributes. The syntax for REQUIRES syntax is always enabled,
and uses the available_features attribute to define what symbols can be used in
REQUIRES.

[jvoung (off chromium), 2014/10/20 17:36:00]

A good example might be tests_lit/llvm2ice_tests/addr-opt-multi-def-var.ll. That
requires some logging from "--verbose addropt" to work. I guess that could also
be done with the %if.

[Karl, 2014/10/20 21:00:42]

Added "REQUIRES: x86-32" to test file, since it is part of the x8632 lowering
code.

 ; RUN: %p2i -i %s --args -O2 --verbose none \
 ; RUN:   | FileCheck --check-prefix=CALLTARGETS %s
 ; RUN: %p2i -i %s --args -O2 --verbose none \