Revert "clang/test/Driver/crash-report.c: This requires rewriter for -frewrite-includes. [PR20321]"

We've decided to make the core rewriter class and PP rewriters mandatory.
They're only a few hundred lines of code in total and not worth supporting as a
distinct build configuration, especially since doing so disables key compiler
features.

This reverts commit r213150.

Revert "clang/test: Introduce the feature "rewriter" for --enable-clang-rewriter."

This reverts commit r213148.

Revert "Move clang/test/Frontend/rewrite-*.c to clang/test/Frontend/Rewriter/"

This reverts commit r213146.

llvm-svn: 213159

clang/test/CXX/basic/basic.start/basic.start.main/p2.cpp

@@ -10,7 +10,6 @@
 // RUN: cp %s %t
 // RUN: %clang_cc1 -x c++ %s -std=c++11 -fsyntax-only -verify -DTEST9
 // RUN: not %clang_cc1 -x c++ %t -std=c++11 -fixit -DTEST9
-// REQUIRES: rewriter
 // RUN: %clang_cc1 -x c++ %t -std=c++11 -fsyntax-only -DTEST9
 
 // RUN: %clang_cc1 -fsyntax-only -verify %s -DTEST10

clang/test/CXX/temp/temp.spec/cxx1y-variable-template-no-body.cpp

@@ -2,7 +2,6 @@
 // RUN: cp %s %t
 // RUN: not %clang_cc1 --std=c++1y -x c++ -fixit %t -DFIXING
 // RUN: %clang_cc1 --std=c++1y -x c++ %t -DFIXING
-// REQUIRES: rewriter
 
 template<typename T> 
 T pi = T(3.1415926535897932385); // expected-note {{template is declared here}}

clang/test/CXX/temp/temp.spec/no-body.cpp

@@ -2,7 +2,6 @@
 // RUN: cp %s %t
 // RUN: not %clang_cc1 -x c++ -fixit %t -DFIXING
 // RUN: %clang_cc1 -x c++ %t -DFIXING
-// REQUIRES: rewriter
 
 template<typename T> void f(T) { }
 template<typename T> void g(T) { }

clang/test/Coverage/html-print.c

@@ -1,6 +1,3 @@
 // RUN: %clang_cc1 -emit-html -o %t %s
 
-// -emit-html requires rewriter.
-// REQUIRES: rewriter
-
 #include "c-language-features.inc"

clang/test/Driver/crash-report.c

@@ -10,9 +10,6 @@
 // RUN: cat %t/crash-report-*.sh | FileCheck --check-prefix=CHECKSH %s
 // REQUIRES: crash-recovery
 
-// PR20321: diag invokes clang -E with -frewrite-includes.
-// REQUIRES: rewriter
-
 // because of the glob (*.c, *.sh)
 // REQUIRES: shell
 

clang/test/Format/lit.local.cfg

@@ -1,2 +0,0 @@
-if config.root.clang_rewriter == 0:
-    config.unsupported = True

clang/test/Frontend/Rewriter/lit.local.cfg

@@ -1,2 +0,0 @@
-if config.root.clang_rewriter == 0:
-    config.unsupported = True

clang/test/Frontend/rewrite-includes-modules.c

@@ -1,5 +1,5 @@
 // RUN: rm -rf %t
-// RUN: %clang_cc1 -fmodules -fmodules-cache-path=%t -x objective-c %s -F %S/../../Modules/Inputs -E -frewrite-includes -o - | FileCheck %s
+// RUN: %clang_cc1 -fmodules -fmodules-cache-path=%t -x objective-c %s -F %S/../Modules/Inputs -E -frewrite-includes -o - | FileCheck %s
 
 int bar();
 #include <Module/Module.h>

clang/test/Misc/emit-html-insert.c

@@ -1,5 +1,4 @@
 // RUN: %clang_cc1 %s -emit-html -o - | grep ">< 10; }"
-// REQUIRES: rewriter
 
 int a(int x) { return x
 < 10; }

clang/test/Misc/emit-html.c

@@ -1,5 +1,4 @@
 // RUN: %clang_cc1 %s -emit-html -o -
-// REQUIRES: rewriter
 
 // rdar://6562329
 #line 42 "foo.c"

clang/test/Parser/brackets.c

@@ -3,7 +3,6 @@
 // RUN: not %clang_cc1 -fixit %t -x c -DFIXIT
 // RUN: %clang_cc1 -fsyntax-only %t -x c -DFIXIT
 // RUN: not %clang_cc1 -fsyntax-only -fdiagnostics-parseable-fixits %s 2>&1 | FileCheck %s -strict-whitespace
-// REQUIRES: rewriter
 
 void test1() {
   int a[] = {0,1,1,2,3};

clang/test/Parser/brackets.cpp

@@ -3,7 +3,6 @@
 // RUN: not %clang_cc1 -fixit %t -x c++ -DFIXIT
 // RUN: %clang_cc1 -fsyntax-only %t -x c++ -DFIXIT
 // RUN: not %clang_cc1 -fsyntax-only -fdiagnostics-parseable-fixits %s 2>&1 | FileCheck %s -strict-whitespace
-// REQUIRES: rewriter
 
 void test1() {
   int a[] = {0,1,1,2,3};

clang/test/Parser/colon-colon-parentheses.cpp

@@ -2,7 +2,6 @@
 // RUN: cp %s %t
 // RUN: not %clang_cc1 -x c++ -fixit %t
 // RUN: %clang_cc1 -x c++ %t
-// REQUIRES: rewriter
 
 struct S { static int a,b,c;};
 int S::(a);  // expected-error{{unexpected parenthesis after '::'}}

clang/test/Parser/cxx-extra-semi.cpp

@@ -4,7 +4,6 @@
 // RUN: cp %s %t
 // RUN: %clang_cc1 -x c++ -Wextra-semi -fixit %t
 // RUN: %clang_cc1 -x c++ -Wextra-semi -Werror %t
-// REQUIRES: rewriter
 
 class A {
   void A1();

clang/test/Parser/nested-namespaces-recovery.cpp

@@ -2,7 +2,6 @@
 // RUN: %clang_cc1 -fsyntax-only -verify %s
 // RUN: not %clang_cc1 -x c++ -fixit %t
 // RUN: %clang_cc1 -x c++ %t
-// REQUIRES: rewriter
 
 namespace foo1::foo2::foo3 { // expected-error {{nested namespace definition must define each namespace separately}}
   int foo(int x) { return x; }

clang/test/Preprocessor/ignore-pragmas.c

@@ -3,7 +3,6 @@
 // RUN: %clang -M -Wall %s -Xclang -verify
 // RUN: %clang -E -frewrite-includes %s -Wall -Xclang -verify
 // RUN: %clang -E -dD -dM %s -Wall -Xclang -verify
-// REQUIRES: rewriter
 // expected-no-diagnostics
 
 #pragma GCC visibility push (default)

clang/test/Sema/format-strings-fixit-ssize_t.c

@@ -2,7 +2,6 @@
 // RUN: %clang_cc1 -triple x86_64-apple-darwin10 -pedantic -Wall -fixit %t
 // RUN: %clang_cc1 -triple x86_64-apple-darwin10 -fsyntax-only -pedantic -Wall -Werror %t
 // RUN: %clang_cc1 -triple x86_64-apple-darwin10 -E -o - %t | FileCheck %s
-// REQUIRES: rewriter
 
 /* This is a test of the various code modification hints that are
    provided as part of warning or extension diagnostics. All of the

clang/test/Sema/format-strings-fixit.c

@@ -2,7 +2,6 @@
 // RUN: %clang_cc1 -pedantic -Wall -fixit %t
 // RUN: %clang_cc1 -fsyntax-only -pedantic -Wall -Werror %t
 // RUN: %clang_cc1 -E -o - %t | FileCheck %s
-// REQUIRES: rewriter
 
 /* This is a test of the various code modification hints that are
    provided as part of warning or extension diagnostics. All of the

clang/test/Sema/format-strings-no-fixit.c

@@ -1,7 +1,6 @@
 // RUN: cp %s %t
 // RUN: %clang_cc1 -fsyntax-only -fixit %t
 // RUN: %clang_cc1 -E -o - %t | FileCheck %s
-// REQUIRES: rewriter
 
 /* This is a test of the various code modification hints that are
    provided as part of warning or extension diagnostics. Only

clang/test/Sema/warn-documentation-almost-trailing.c

@@ -3,7 +3,6 @@
 // RUN: cp %s %t
 // RUN: %clang_cc1 -fsyntax-only -Wdocumentation -fixit %t
 // RUN: %clang_cc1 -fsyntax-only -Wdocumentation -Werror %t
-// REQUIRES: rewriter
 
 struct a {
   int x; //< comment // expected-warning {{not a Doxygen trailing comment}}

clang/test/Sema/warn-strncat-size.c

@@ -2,7 +2,6 @@
 // RUN: %clang_cc1 -DUSE_BUILTINS -Wstrncat-size -verify -fsyntax-only %s
 // RUN: %clang_cc1 -fsyntax-only -Wstrncat-size -fixit -x c %s
 // RUN: %clang_cc1 -DUSE_BUILTINS -fsyntax-only -Wstrncat-size -fixit -x c %s
-// REQUIRES: rewriter
 
 typedef __SIZE_TYPE__ size_t;
 size_t strlen (const char *s);

clang/test/SemaCXX/delete.cpp

@@ -2,7 +2,6 @@
 // RUN: cp %s %t
 // RUN: %clang_cc1 -fixit -x c++ %t
 // RUN: %clang_cc1 -E -o - %t | FileCheck %s
-// REQUIRES: rewriter
 
 void f(int a[10][20]) {
   // CHECK: delete[] a;

clang/test/SemaObjC/cocoa-api-usage.m

@@ -3,7 +3,6 @@
 // RUN: cp %s %t.m
 // RUN: %clang_cc1 -triple x86_64-apple-darwin10 -fobjc-arc %t.m -fixit -Wobjc-cocoa-api
 // RUN: diff %s.fixed %t.m
-// REQUIRES: rewriter
 
 typedef signed char BOOL;
 #define nil ((void*) 0)

clang/test/SemaObjC/cocoa-api-usage.m.fixed

@@ -3,7 +3,6 @@
 // RUN: cp %s %t.m
 // RUN: %clang_cc1 -triple x86_64-apple-darwin10 -fobjc-arc %t.m -fixit -Wobjc-cocoa-api
 // RUN: diff %s.fixed %t.m
-// REQUIRES: rewriter
 
 typedef signed char BOOL;
 #define nil ((void*) 0)

clang/test/lit.cfg

@@ -335,9 +335,6 @@ for pattern in [r"\bFileCheck\b",
 # Set available features we allow tests to conditionalize on.
 #
 # Enabled/disabled features
-if config.clang_rewriter != 0:
-    config.available_features.add("rewriter")
-
 if config.clang_staticanalyzer != 0:
     config.available_features.add("staticanalyzer")