[libunwind][AIX] Mark signal_frame.pass.cpp UNSUPPORTED on AIX

[xingxue]

Summary

This patch marks libunwind test case signal_frame.pass.cpp as UNSUPPORTED on AIX because the AIX assembler does not support CFI directives.

[MaskRay]

LGTM.

libunwind/test/signal_frame.pass.cpp

18

I assume that you have checked that target=powerpc{{|64}}-ibm-aix work with both UNSUPPORTED and REQUIRES.

[ldionne]

LGTM with suggestions.

libunwind/test/signal_frame.pass.cpp

18

Yes it does -- it's just a normal Lit feature.

Please add a comment explaining:

// The AIX assembler does not support CFI directives, which
// are necessary to run this test.
// UNSUPPORTED: target=powerpc{{(64)?}}-ibm-aix

I also suggest using (64)? instead of |64 for the regex. It seems more idiomatic to me.

[xingxue]

Thanks for reviewing this patch, @danielkiss, @MaskRay, @ldionne! Yes, target=powerpc-ibm-aix (32-bit) and target=powerpc64-ibm-aix (64-bit) are LIT features. Updated the patch as suggested. Thanks, @ldionne!

libunwind/test/signal_frame.pass.cpp

 // Ensure that functions marked as signal frames are reported as such.
 
 // TODO: Investigate these failures
 // XFAIL: asan, tsan, ubsan
 
 // TODO: Investigate this failure on macOS
 // XFAIL: target={{.+}}-apple-darwin{{.+}}
 
 // UNSUPPORTED: libunwind-arm-ehabi
 
+// The AIX assembler does not support CFI directives, which
+// are necessary to run this test.
+// UNSUPPORTED: target=powerpc{{(64)?}}-ibm-aix
+
 #include <assert.h>
 #include <stdlib.h>
 #include <libunwind.h>
 
 void test() {
   asm(".cfi_signal_frame");
   unw_cursor_t cursor;
   unw_context_t uc;