From 2e1538f282802873e5b5b3a4406055802545bf98 Mon Sep 17 00:00:00 2001 From: Reid Kleckner Date: Thu, 20 Oct 2016 23:11:45 +0000 Subject: Remove 24 instances of 'REQUIRES: shell' Tests fall into one of the following categories: - The requirement was unnecessary - Additional quoting was required for backslashes in paths (see "sed -e 's/\\/\\\\/g'") in the sanitizer tests. - OpenMP used 'REQUIRES: shell' as a proxy for the test failing on Windows. Those tests fail there reliably, so use XFAIL instead. I tried not to remove shell requirements that were added to suppress flaky test failures, but if I screwed up, we can add it back as needed. llvm-svn: 284793 --- clang/test/OpenMP/task_firstprivate_codegen.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'clang/test/OpenMP/task_firstprivate_codegen.cpp') diff --git a/clang/test/OpenMP/task_firstprivate_codegen.cpp b/clang/test/OpenMP/task_firstprivate_codegen.cpp index 0d8e1c4afe4..b17729312a2 100644 --- a/clang/test/OpenMP/task_firstprivate_codegen.cpp +++ b/clang/test/OpenMP/task_firstprivate_codegen.cpp @@ -7,7 +7,7 @@ // expected-no-diagnostics // REQUIRES: x86-registered-target // It doesn't pass on win32. -// REQUIRES: shell +// XFAIL: win32 #ifndef ARRAY #ifndef HEADER #define HEADER -- cgit v1.2.3