Subzero, MIPS32: Remove --skip-unimplemented from lit tests

tests_lit/llvm2ice_tests/nacl-other-intrinsics.ll

Index: tests_lit/llvm2ice_tests/nacl-other-intrinsics.ll
diff --git a/tests_lit/llvm2ice_tests/nacl-other-intrinsics.ll b/tests_lit/llvm2ice_tests/nacl-other-intrinsics.ll
index eacce418d447a39d0be8bc988e0c3899c141d737..684f4992e8bf33125985f59bd93904cd654b5a15 100644
--- a/tests_lit/llvm2ice_tests/nacl-other-intrinsics.ll
+++ b/tests_lit/llvm2ice_tests/nacl-other-intrinsics.ll
@@ -36,7 +36,7 @@
 
 ; RUN: %if --need=target_MIPS32 --need=allow_dump \
 ; RUN:   --command %p2i --filetype=asm --assemble --disassemble --target mips32\
-; RUN:   -i %s --args -Om1 --skip-unimplemented \
+; RUN:   -i %s --args -Om1 \

[Jim Stichnoth, 2016/10/25 14:22:19]

Unfortunately, this --skip-unimplemented may need to stay.
  make -f Makefile.standalone check-lit
fails (at least on my machine) because of the unimplemented stacksave and
stackrestore intrinsics.

These tests don't currently have any MIPS32 check lines, so it may seem odd that
they cause the test to fail.  I think there are two possibilities:

1. The unimplemented error message gets mixed in with the filetype=asm text
stream and makes the llvm-mc assembler fail.

2. Because of pnacl-sz's default -threads=2 setting, one thread could hit the
unimplemented error and cause an abort before another thread produces sufficient
output for the other tests.

Note that there are still a bunch of other instances of
UnimplementedLoweringError in the MIPS lowering that may not be tested by the
lit tests.  (and a few in the ARM lowering, which need investigation...)

I tried to draw out more failures of this type by changing the default
-threads=2 to -threads=100 (via IceClFlags.def) and didn't find any more, but
they could still be lurking...

 ; RUN:   -allow-externally-defined-symbols \
 ; RUN:   | %if --need=target_MIPS32 --need=allow_dump \
 ; RUN:   --command FileCheck --check-prefix MIPS32 %s