Mark a test as requiring a shell

[• rafael]

Summary

These two have hard to replace uses of sub shells.

[rnk]

compiler-rt/test/asan/TestCases/Linux/read_binary_name_regtest.c

5–6

I feel like a more LLVM-y way to do this would be to feature test seccomp in cmake and use REQUIRES: seccomp or something, but it's totally not worth it for one test.

compiler-rt/test/asan/TestCases/Linux/swapcontext_annotation.cc

7 ↗

(On Diff #94430)

I think we can just replace this with a temporary file:

// RUN: seq 60 | xargs -i -- grep LOOPCHECK %s > %t.checks
// RUN: %clangxx_asan ... | FileCheck %t.checks --check-prefix=LOOPCHECK
// RUN: %clangxx_asan ... | FileCheck %t.checks --check-prefix=LOOPCHECK
// RUN: %clangxx_asan ... | FileCheck %t.checks --check-prefix=LOOPCHECK

That's more or less what <() is doing.

[rnk]

lgtm

[• rafael]

r301265

compiler-rt/test/asan/TestCases/Linux/read_binary_name_regtest.c

 // Regression test for https://crbug.com/502974, where ASan was unable to read
 // the binary name because of sandbox restrictions.
 // This test uses seccomp-BPF to restrict the readlink() system call and makes
 // sure ASan is still able to
 // RUN: not ls /usr/include/linux/seccomp.h || ( %clang_asan %s -o %t && not %run %t 2>&1 | FileCheck %s )
+// REQUIRES: shell
 // UNSUPPORTED: android
 
 #include <errno.h>
 #include <stddef.h>
 #include <stdlib.h>
 #include <stdio.h>
 #include <sys/prctl.h>
 #include <sys/syscall.h>