]> granicus.if.org Git - clang/commitdiff
Re-land Remove REQUIRES:shell from tests that pass for me on Windows
authorReid Kleckner <rnk@google.com>
Tue, 10 Sep 2019 20:15:45 +0000 (20:15 +0000)
committerReid Kleckner <rnk@google.com>
Tue, 10 Sep 2019 20:15:45 +0000 (20:15 +0000)
This reverts r371497 (git commit 3d7e9ab7b9f8c53aa41420c54970f0fb421004a2)

Reorder `not` with `env` in these two tests so they pass:
  Driver/rewrite-map-in-diagnostics.c
  Index/crash-recovery-modules.m.

This will not be necessary after D66531 lands.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@371552 91177308-0d34-0410-b5e6-96231b3b80d8

25 files changed:
test/Analysis/crash-trace.c
test/CodeGen/thinlto_backend.ll
test/Driver/check-time-trace-sections.cpp
test/Driver/check-time-trace.cpp
test/Driver/clang-offload-bundler.c
test/Driver/crash-report-crashfile.m
test/Driver/rewrite-map-in-diagnostics.c
test/Format/style-on-command-line.cpp
test/Frontend/dependency-gen-has-include.c
test/Index/crash-recovery-modules.m
test/Modules/at-import-in-framework-header.m
test/Modules/builtins.m
test/Modules/dependency-dump-dependent-module.m
test/Modules/dependency-dump.m
test/Modules/implicit-invalidate-common.c
test/OpenMP/task_firstprivate_codegen.cpp
test/OpenMP/task_private_codegen.cpp
test/OpenMP/taskloop_firstprivate_codegen.cpp
test/OpenMP/taskloop_lastprivate_codegen.cpp
test/OpenMP/taskloop_private_codegen.cpp
test/OpenMP/taskloop_simd_firstprivate_codegen.cpp
test/OpenMP/taskloop_simd_lastprivate_codegen.cpp
test/OpenMP/taskloop_simd_private_codegen.cpp
test/PCH/modified-header-error.c
test/Parser/crash-report.c

index ef1763ef97a55bcd251bf0d21ee8d6f7d45fa932..0c8c8cc6cb2e97a028b977df063f97d49c978651 100644 (file)
@@ -1,9 +1,8 @@
 // RUN: not --crash %clang_analyze_cc1 -analyzer-checker=debug.ExprInspection %s 2>&1 | FileCheck %s
 // REQUIRES: crash-recovery
 
-// FIXME: CHECKs might be incompatible to win32.
-// Stack traces also require back traces.
-// REQUIRES: shell, backtrace
+// Stack traces require back traces.
+// REQUIRES: backtrace
 
 void clang_analyzer_crash(void);
 
@@ -18,6 +17,6 @@ void test() {
 // CHECK: 0.   Program arguments: {{.*}}clang
 // CHECK-NEXT: 1.      <eof> parser at end of file
 // CHECK-NEXT: 2. While analyzing stack: 
-// CHECK-NEXT:  #0 Calling inlined at line 15
+// CHECK-NEXT:  #0 Calling inlined at line 14
 // CHECK-NEXT:  #1 Calling test
 // CHECK-NEXT: 3.      {{.*}}crash-trace.c:{{[0-9]+}}:3: Error evaluating statement
index 2dd919d5f7de4fcf0af663a72f6464dba1d4a9e1..fcfc7ea46657fc5703497d42e6efd55ca3700bd0 100644 (file)
@@ -1,5 +1,4 @@
-; shell required since the windows bot does not like the "(cd ..."
-; REQUIRES: x86-registered-target, shell
+; REQUIRES: x86-registered-target
 
 ; RUN: opt -module-summary -o %t1.o %s
 ; RUN: opt -module-summary -o %t2.o %S/Inputs/thinlto_backend.ll
 ; RUN: %clang -target x86_64-unknown-linux-gnu -O2 -o %t3.o -x ir %t1.o -c -fthinlto-index=%t.thinlto.bc -save-temps=obj
 ; RUN: llvm-dis %t1.s.3.import.bc -o - | FileCheck --check-prefix=CHECK-IMPORT %s
 ; RUN: mkdir -p %T/dir1
-; RUN: (cd %T/dir1 && %clang -target x86_64-unknown-linux-gnu -O2 -o %t3.o -x ir %t1.o -c -fthinlto-index=%t.thinlto.bc -save-temps=cwd)
+; RUN: cd %T/dir1
+; RUN: %clang -target x86_64-unknown-linux-gnu -O2 -o %t3.o -x ir %t1.o -c -fthinlto-index=%t.thinlto.bc -save-temps=cwd
+; RUN: cd ../..
 ; RUN: llvm-dis %T/dir1/*1.s.3.import.bc -o - | FileCheck --check-prefix=CHECK-IMPORT %s
 ; RUN: mkdir -p %T/dir2
-; RUN: (cd %T/dir2 && %clang -target x86_64-unknown-linux-gnu -O2 -o %t3.o -x ir %t1.o -c -fthinlto-index=%t.thinlto.bc -save-temps)
+; RUN: cd %T/dir2
+; RUN: %clang -target x86_64-unknown-linux-gnu -O2 -o %t3.o -x ir %t1.o -c -fthinlto-index=%t.thinlto.bc -save-temps
+; RUN: cd ../..
 ; RUN: llvm-dis %T/dir2/*1.s.3.import.bc -o - | FileCheck --check-prefix=CHECK-IMPORT %s
 ; CHECK-IMPORT: define available_externally void @f2()
 ; RUN: llvm-nm %t3.o | FileCheck --check-prefix=CHECK-OBJ %s
index 4d1c0fde3a8cd044466a5ebecdfd0e21f38647f9..2dbe2037550a5843617c54eaca9be4f75ab41040 100644 (file)
@@ -1,4 +1,3 @@
-// REQUIRES: shell
 // RUN: %clangxx -S -ftime-trace -ftime-trace-granularity=0 -o %T/check-time-trace-sections %s
 // RUN: cat %T/check-time-trace-sections.json | %python %S/check-time-trace-sections.py
 
index 9d381786b1877a9d6d846486d04e3ff00550bcf5..3c6a002ae8ab9c9ea25e9a9e5b45bda947c8e75e 100644 (file)
@@ -1,4 +1,3 @@
-// REQUIRES: shell
 // RUN: %clangxx -S -ftime-trace -ftime-trace-granularity=0 -o %T/check-time-trace %s
 // RUN: cat %T/check-time-trace.json \
 // RUN:   | %python -c 'import json, sys; json.dump(json.loads(sys.stdin.read()), sys.stdout, sort_keys=True, indent=2)' \
index 44366920fe2529efc936d3b42b2314fc7a68cdd6..be17b092ef34192997c8a74d9a26cbf82eac1aca 100644 (file)
@@ -1,7 +1,6 @@
 // REQUIRES: x86-registered-target
 // REQUIRES: powerpc-registered-target
-// REQUIRES: shell
-// UNSUPPORTED: ms-sdk, darwin
+// UNSUPPORTED: darwin
 
 //
 // Generate all the types of files we can bundle.
index fbfb532703efeb5eebedf7ef8db30976b691c800..bdfac327516fe76a9a86d11970dd851f0b5a5f6c 100644 (file)
@@ -1,4 +1,4 @@
-// REQUIRES: crash-recovery, shell, system-darwin
+// REQUIRES: crash-recovery, system-darwin
 // RUN: rm -rf %t
 // RUN: mkdir -p %t/i %t/m %t
 
index 4aea0ec031456bdb8de3f32898dd9a77c864e119..ef598ff66b841c8025cd86853cf4a92bb785da4b 100644 (file)
@@ -1,7 +1,7 @@
 // RUN: rm -rf "%t"
 // RUN: mkdir -p "%t"
-// RUN: not env TMPDIR="%t" TEMP="%t" TMP="%t" RC_DEBUG_OPTION=1 \
-// RUN:     %clang -fsyntax-only -frewrite-map-file %p/Inputs/rewrite.map %s 2>&1 \
+// RUN: env TMPDIR="%t" TEMP="%t" TMP="%t" RC_DEBUG_OPTION=1 \
+// RUN:     not %clang -fsyntax-only -frewrite-map-file %p/Inputs/rewrite.map %s 2>&1 \
 // RUN:   | FileCheck %s
 
 #pragma clang __debug parser_crash
@@ -9,6 +9,3 @@
 // CHECK: note: diagnostic msg: {{.*}}rewrite.map
 
 // REQUIRES: crash-recovery
-
-// FIXME: This doesn't fail on "env clang". Investigating.
-// REQUIRES: shell
index 0e6078e2fd67696e0585062616be53e38154dc0a..ba06babee35a80049e7fa97c7ce7ab9044cf90ef 100644 (file)
@@ -39,4 +39,4 @@ int j;
 
 // On Windows, the 'rm' commands fail when the previous process is still alive.
 // This happens enough to make the test useless.
-// REQUIRES: shell
+// UNSUPPORTED: system-windows
index e8abb2c9c163bf13b90645d531854d2c535c3870..51de72a5a0fb1c18a2c3951019f162a114e5045a 100644 (file)
@@ -1,5 +1,3 @@
-// REQUIRES: shell
-
 // Basic test
 // RUN: rm -rf %t.dir
 // RUN: mkdir %t.dir
 // RUN: FileCheck -input-file=%t.dir/file.deps %s
 // CHECK: dependency-gen-has-include.o
 // CHECK: dependency-gen-has-include.c
-// CHECK: a/header.h
-// CHECK-NOT: missing/file.h
-// CHECK: system/system-header.h
-// CHECK: next-a/next-header.h
-// CHECK: next-b/next-header.h
+// CHECK: a{{[/\\]}}header.h
+// CHECK-NOT: missing{{[/\\]}}file.h
+// CHECK: system{{[/\\]}}system-header.h
+// CHECK: next-a{{[/\\]}}next-header.h
+// CHECK: next-b{{[/\\]}}next-header.h
 
 // Verify that we ignore system headers in user-only headers mode.
 // RUN: %clang -MMD -MF %t.dir/user-headers.deps %s -fsyntax-only -I %t.dir -isystem %t.dir/system -I %t.dir/next-a -I %t.dir/next-b
 // RUN: FileCheck -input-file=%t.dir/user-headers.deps --check-prefix CHECK-USER-HEADER %s
-// CHECK-USER-HEADER-NOT: system/system-header.h
+// CHECK-USER-HEADER-NOT: system{{[/\\]}}system-header.h
 
 #if __has_include("a/header.h")
 #endif
index 296416df97f28db2a9cf0af96dc9e7ed12c67807..57d1cf7d0ec74655dd1d903be0c0c58ead1cf3e7 100644 (file)
@@ -2,16 +2,15 @@
 // RUN: rm -rf %t
 
 // Parse the file, such that building the module will cause Clang to crash.
-// RUN: not env CINDEXTEST_FAILONERROR=1 c-index-test -test-load-source all -fmodules -fmodules-cache-path=%t -Xclang -fdisable-module-hash -I %S/Inputs/Headers -DCRASH %s > /dev/null 2> %t.err
+// RUN: env CINDEXTEST_FAILONERROR=1 not c-index-test -test-load-source all -fmodules -fmodules-cache-path=%t -Xclang -fdisable-module-hash -I %S/Inputs/Headers -DCRASH %s > /dev/null 2> %t.err
 // RUN: FileCheck < %t.err -check-prefix=CHECK-CRASH %s
-// CHECK-CRASH: crash-recovery-modules.m:17:9:{17:2-17:14}: fatal error: could not build module 'Crash'
+// CHECK-CRASH: crash-recovery-modules.m:16:9:{16:2-16:14}: fatal error: could not build module 'Crash'
 
 // Parse the file again, without crashing, to make sure that
 // subsequent parses do the right thing.
 // RUN: env CINDEXTEST_FAILONERROR=1 c-index-test -test-load-source all -fmodules -fmodules-cache-path=%t -Xclang -fdisable-module-hash -I %S/Inputs/Headers %s > /dev/null
 
 // REQUIRES: crash-recovery
-// REQUIRES: shell
 // UNSUPPORTED: libstdcxx-safe-mode
 
 @import Crash;
index fe3663812075db104458a08c9a0cfe7b0a84f292..043c69462edf3cb0c376f7d444859b758c4dff31 100644 (file)
@@ -1,5 +1,3 @@
-// REQUIRES: shell
-
 // RUN: rm -rf %t
 // RUN: mkdir %t
 
index 88a44e7b0aa345c06cc1a9f6ea9d8940ebe177bd..2480e6379cc85900e681f3c0b22fe339f422e829 100644 (file)
@@ -8,7 +8,6 @@
 // RUN: %clang_cc1 -fmodules-cache-path=%t.pch.cache -fmodules -fimplicit-module-maps -I %S/Inputs %s -include-pch %t.pch %s -verify
 
 // expected-no-diagnostics
-// REQUIRES: shell
 
 void use_constant_string_builtins1(void) {
   (void)__builtin___CFStringMakeConstantString("");
index 49069861654654113adf2ef2014bd5f75427baa2..2430726d3ae5a1f5ee38912c63d49ca742a6d5ab 100644 (file)
@@ -1,8 +1,6 @@
 // When a module depends on another, check that we dump the dependency header
 // files for both.
 
-// REQUIRES: shell
-
 // RUN: rm -rf %t
 // RUN: %clang_cc1 -fmodules -fimplicit-module-maps -fmodules-cache-path=%t/cache -module-dependency-dir %t/vfs -F %S/Inputs -I %S/Inputs -verify %s
 // expected-no-diagnostics
index deb66188e14727a0760d2e8a5a50ff4736eb7378..f3a487544b8e9e2f984a34647171184b7fe3d505 100644 (file)
@@ -1,8 +1,6 @@
 // Check that we can dump all of the headers a module depends on, and a VFS map
 // for the same.
 
-// REQUIRES: shell
-
 // RUN: rm -rf %t
 // RUN: %clang_cc1 -fmodules -fimplicit-module-maps -fmodules-cache-path=%t/cache -module-dependency-dir %t/vfs -F %S/Inputs -I %S/Inputs -verify %s
 // expected-no-diagnostics
index d6cf90e12dc307105518fedd9f5e045f4b7c001a..143a00ff37a8ebe0e8888db27cc4913d595b76c3 100644 (file)
@@ -1,4 +1,3 @@
-// REQUIRES: shell
 // RUN: rm -rf %t
 // RUN: mkdir -p %t/implicit-invalidate-common
 // RUN: cp -r %S/Inputs/implicit-invalidate-common %t/
index 0517ec145be533a4d6303c969a98278637468af7..e9c6d6d738523dfad174937a5635c5d60f997976 100644 (file)
@@ -13,8 +13,6 @@
 // RUN: %clang_cc1 -verify -fopenmp-simd -x c++ -std=c++11 -DARRAY -triple x86_64-apple-darwin10 -emit-llvm %s -o - | FileCheck --check-prefix SIMD-ONLY0 %s
 // SIMD-ONLY0-NOT: {{__kmpc|__tgt}}
 // expected-no-diagnostics
-// It doesn't pass on win32.
-// REQUIRES: shell
 #ifndef ARRAY
 #ifndef HEADER
 #define HEADER
index 17b0de16a1c3fb9113b7efd4ab49dd913ff08ccd..32303cd600a7239e9a67e1f6fb42ce999fdf4741 100644 (file)
@@ -13,8 +13,6 @@
 // RUN: %clang_cc1 -verify -fopenmp-simd -x c++ -std=c++11 -DARRAY -triple x86_64-apple-darwin10 -emit-llvm %s -o - | FileCheck --check-prefix SIMD-ONLY0 %s
 // SIMD-ONLY0-NOT: {{__kmpc|__tgt}}
 // expected-no-diagnostics
-// It doesn't pass on win32. Investigating.
-// REQUIRES: shell
 
 #ifndef ARRAY
 #ifndef HEADER
index 0aa9f3e6966ad277df558621d69d5f3eeed2ccef..64723814abb5777b4400ed1f9ac226c9ddefdf61 100644 (file)
@@ -13,8 +13,7 @@
 // RUN: %clang_cc1 -verify -fopenmp-simd -x c++ -std=c++11 -DARRAY -triple x86_64-apple-darwin10 -emit-llvm %s -o - | FileCheck --check-prefix SIMD-ONLY0 %s
 // SIMD-ONLY0-NOT: {{__kmpc|__tgt}}
 // expected-no-diagnostics
-// It doesn't pass on win32.
-// REQUIRES: shell
+
 #ifndef ARRAY
 #ifndef HEADER
 #define HEADER
index 6e6588571103514e86213c50a98d9162d877433f..1df56ac9fdaef77f41c05d8871583a915c088dd0 100644 (file)
@@ -15,8 +15,7 @@
 // RUN: %clang_cc1 -verify -fopenmp-simd -x c++ -std=c++11 -DLOOP -triple x86_64-apple-darwin10 -emit-llvm %s -o - | FileCheck -check-prefix=SIMD-ONLY0 %s
 // SIMD-ONLY0-NOT: {{__kmpc|__tgt}}
 // expected-no-diagnostics
-// It doesn't pass on win32.
-// REQUIRES: shell
+
 #if !defined(ARRAY) && !defined(LOOP)
 #ifndef HEADER
 #define HEADER
index c7a85a4661a0f1bf7f0c02176de6f99037acebf3..01c387e95d7e02262cdfdcca9ba44ad9826b2b9a 100644 (file)
@@ -13,8 +13,6 @@
 // RUN: %clang_cc1 -verify -fopenmp-simd -x c++ -std=c++11 -DARRAY -triple x86_64-apple-darwin10 -emit-llvm %s -o - | FileCheck --check-prefix SIMD-ONLY0 %s
 // SIMD-ONLY0-NOT: {{__kmpc|__tgt}}
 // expected-no-diagnostics
-// It doesn't pass on win32. Investigating.
-// REQUIRES: shell
 
 #ifndef ARRAY
 #ifndef HEADER
index 5f0b5dea1f9ea0444826f7c39957d3084fb7a07b..935aacc77b5941abb159736f3f8bcbf2816498d5 100644 (file)
@@ -13,8 +13,7 @@
 // RUN: %clang_cc1 -verify -fopenmp-simd -x c++ -std=c++11 -DARRAY -triple x86_64-apple-darwin10 -emit-llvm %s -o - | FileCheck --check-prefix SIMD-ONLY0 %s
 // SIMD-ONLY0-NOT: {{__kmpc|__tgt}}
 // expected-no-diagnostics
-// It doesn't pass on win32.
-// REQUIRES: shell
+
 #ifndef ARRAY
 #ifndef HEADER
 #define HEADER
index 051c843e0a61c48f878853b9550477b30ce2f46f..97007dde2f2b7a26d8c066f593cf31c027739286 100644 (file)
@@ -13,8 +13,7 @@
 // RUN: %clang_cc1 -verify -fopenmp-simd -x c++ -std=c++11 -DARRAY -triple x86_64-apple-darwin10 -emit-llvm %s -o - | FileCheck --check-prefix SIMD-ONLY0 %s
 // SIMD-ONLY0-NOT: {{__kmpc|__tgt}}
 // expected-no-diagnostics
-// It doesn't pass on win32.
-// REQUIRES: shell
+
 #ifndef ARRAY
 #ifndef HEADER
 #define HEADER
index 2425d23dbaf1030bc1fe25123f903d4496e8a69e..29b9c8302be3b277a2fe360b4c0a2ac6b6d29271 100644 (file)
@@ -13,8 +13,6 @@
 // RUN: %clang_cc1 -verify -fopenmp-simd -x c++ -std=c++11 -DARRAY -triple x86_64-apple-darwin10 -emit-llvm %s -o - | FileCheck --check-prefix SIMD-ONLY0 %s
 // SIMD-ONLY0-NOT: {{__kmpc|__tgt}}
 // expected-no-diagnostics
-// It doesn't pass on win32. Investigating.
-// REQUIRES: shell
 
 #ifndef ARRAY
 #ifndef HEADER
index 8ab38e1e9e9843469684040cb9d782d47fd2cd17..fb638363ba659274c8be4f8cef05064676ce1b72 100644 (file)
@@ -9,4 +9,6 @@
 #include "header2.h"
 
 // CHECK: fatal error: file {{.*}} has been modified since the precompiled header {{.*}} was built
-// REQUIRES: shell
+
+// FIXME: Flaky on Windows, timestamp resolution?
+// UNSUPPORTED: win32
index ef65b88a06e7f9943f1788817781dc63288ce433..6ca10459d9a5bece70142599afe241b07f630294 100644 (file)
@@ -1,9 +1,8 @@
 // RUN: not --crash %clang_cc1 %s 2>&1 | FileCheck %s
 // REQUIRES: crash-recovery
 
-// FIXME: CHECKs might be incompatible to win32.
 // Stack traces also require back traces.
-// REQUIRES: shell, backtrace
+// REQUIRES: backtrace
 
 #prag\
 ma clang __debug crash