Skip to content

Commit a348588

Browse files
authored
[clang-tools-extra] Remove 'REQUIRES: shell' from lit tests. (#156950)
As preparation for making lit use the internal shell by default in clang-tools-extra (with significant expected performance gains), this removes 'REQUIRES: shell' from the clang-tools-extra lit tests that have it, and updates the one test that was not passing with 'LIT_USE_INTERNAL_SHELL=1' to now pass with the internal shell.
1 parent 3b452e0 commit a348588

File tree

6 files changed

+7
-9
lines changed

6 files changed

+7
-9
lines changed

clang-tools-extra/test/clang-include-fixer/multiple_fixes.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
// REQUIRES: shell
21
// RUN: sed -e 's#//.*$##' %s > %t.cpp
32
// RUN: mkdir -p %t.dir/clang-include-fixer/multiple-fixes
43
// RUN: echo 'foo f;' > %t.dir/clang-include-fixer/multiple-fixes/foo.cpp

clang-tools-extra/test/clang-tidy/infrastructure/clang-tidy-diff.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// REQUIRES: shell
1+
// UNSUPPORTED: system-windows
22
// RUN: sed 's/placeholder_for_f/f/' %s > %t.cpp
33
// RUN: clang-tidy -checks=-*,modernize-use-override %t.cpp -- -std=c++11 | FileCheck -check-prefix=CHECK-SANITY %s
44
// RUN: not diff -U0 %s %t.cpp | %clang_tidy_diff -checks=-*,modernize-use-override -- -std=c++11 2>&1 | FileCheck %s --check-prefixes=CHECK,CHECK-JMAX

clang-tools-extra/test/clang-tidy/infrastructure/file-filter-symlinks.cpp

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
1-
// REQUIRES: shell
2-
31
// RUN: rm -rf %t
42
// RUN: mkdir -p %t/dir1/dir2
53
// RUN: echo 'class A { A(int); };' > %t/dir1/header.h

clang-tools-extra/test/clang-tidy/infrastructure/header-filter-from-config-file/inheritance/subfolder/bar.cpp

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
1-
// shell is required for the "dirname" command
2-
// REQUIRES: shell
3-
// RUN: clang-tidy -checks=-*,google-explicit-constructor %s -- -I "$(dirname %S)" 2>&1 | FileCheck %s
1+
// UNSUPPORTED: system-windows
2+
// RUN: pushd %S
3+
// RUN: cd ..
4+
// RUN: clang-tidy -checks=-*,google-explicit-constructor %s -- -I "." 2>&1 | FileCheck %s
5+
// RUN: popd
46
#include "foo.h"
57
// CHECK-NOT: foo.h:1:12: warning: single-argument constructors must be marked explicit
68

clang-tools-extra/test/clang-tidy/infrastructure/pr37091.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
// REQUIRES: shell
21
// RUN: rm -rf %t
32
// RUN: mkdir -p %t
43

clang-tools-extra/test/clang-tidy/infrastructure/vfsoverlay.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1+
// UNSUPPORTED: system-windows
12
// RUN: sed -e "s:INPUT_DIR:%S/Inputs/vfsoverlay:g" -e "s:OUT_DIR:%t:g" %S/Inputs/vfsoverlay/vfsoverlay.yaml > %t.yaml
23
// RUN: clang-tidy %s -checks='-*,modernize-use-nullptr' -vfsoverlay %t.yaml -- -I %t | FileCheck %s
3-
// REQUIRES: shell
44

55
#include "not_real.h"
66

0 commit comments

Comments
 (0)