extra test modifications for D34158

[mibintc]

Summary

When adding support for D34158 which changes preprocessed output, I needed to make tiny test corrections for these. Adding the option -ffreestanding suppresses the new behavior, and that's the change I made to fix the tests.

[mibintc]

With the latest proposed fix for D34158, a few more test corrections are needed. The correction consists of suppressing the new preprocessor behavior. D34158 preincludes the file <stdc-predef.h> if the file is available.

[mibintc]

I need to redo the test changes. Will resubmit the diff.

[mibintc]

This patch corresponds to these modifications to clang: https://reviews.llvm.org/D34158?id=105904

[mibintc]

Updating this patch to latest revision of tools/extra

[mibintc]

Updating to latest version. No change other than that.

[mibintc]

Updated the test diff to correspond to the D34158 patch uploaded today.

tools/extra/test/clang-tidy/llvm-include-order.cpp

-// RUN: %check_clang_tidy %s llvm-include-order %t -- -- -isystem %S/Inputs/Headers
+// RUN: %check_clang_tidy %s llvm-include-order %t -- -- -isystem %S/Inputs/Headers -nostdinc
 
 // CHECK-MESSAGES: [[@LINE+2]]:1: warning: #includes are not sorted properly
 #include "j.h"
 #include "gtest/foo.h"
 #include "i.h"
 #include <s.h>
 #include "llvm/a.h"
 #include "clang/b.h"

tools/extra/test/clang-tidy/misc-move-constructor-init.cpp

 // RUN: %check_clang_tidy %s misc-move-constructor-init,modernize-pass-by-value %t -- \
 // RUN: -config='{CheckOptions: \
 // RUN:  [{key: modernize-pass-by-value.ValuesOnly, value: 1}]}' \
-// RUN: -- -std=c++11 -isystem %S/Inputs/Headers
+// RUN: -- -std=c++11 -isystem %S/Inputs/Headers -nostdinc
 
 #include <s.h>
 
 // CHECK-FIXES: #include <utility>
 
 template <class T> struct remove_reference      {typedef T type;};
 template <class T> struct remove_reference<T&>  {typedef T type;};
 template <class T> struct remove_reference<T&&> {typedef T type;};

tools/extra/test/clang-tidy/misc-unconventional-assign-operator.cpp

-// RUN: %check_clang_tidy %s misc-unconventional-assign-operator %t -- -- -std=c++11 -isystem %S/Inputs/Headers -fno-delayed-template-parsing
+// RUN: %check_clang_tidy %s misc-unconventional-assign-operator %t -- -- -std=c++11 -isystem %S/Inputs/Headers -fno-delayed-template-parsing -nostdinc
 
 namespace std {
 template <typename T>
 struct remove_reference { typedef T type; };
 template <typename T>
 struct remove_reference<T &> { typedef T type; };
 template <typename T>
 struct remove_reference<T &&> { typedef T type; };

tools/extra/test/clang-tidy/misc-unused-using-decls.cpp

-// RUN: %check_clang_tidy %s misc-unused-using-decls %t -- -- -fno-delayed-template-parsing -isystem %S/Inputs/
+// RUN: %check_clang_tidy %s misc-unused-using-decls %t -- -- -fno-delayed-template-parsing -isystem %S/Inputs/ -nostdinc
 
 
 // ----- Definitions -----
 template <typename T> class vector {};
 namespace n {
 class A;
 class B;
 class C;

tools/extra/test/clang-tidy/modernize-deprecated-headers-cxx03.cpp

-// RUN: %check_clang_tidy %s modernize-deprecated-headers %t -- -extra-arg-before=-isystem%S/Inputs/modernize-deprecated-headers -- -std=c++03 -v
+// RUN: %check_clang_tidy %s modernize-deprecated-headers %t -- -extra-arg-before=-isystem%S/Inputs/modernize-deprecated-headers -- -std=c++03 -v -nostdinc
 
 #include <assert.h>
 // CHECK-MESSAGES: :[[@LINE-1]]:10: warning: inclusion of deprecated C++ header 'assert.h'; consider using 'cassert' instead [modernize-deprecated-headers]
 // CHECK-FIXES: {{^}}#include <cassert>{{$}}
 #include <complex.h>
 // CHECK-MESSAGES: :[[@LINE-1]]:10: warning: inclusion of deprecated C++ header 'complex.h'; consider using 'complex' instead
 // CHECK-FIXES: {{^}}#include <complex>{{$}}
 #include <ctype.h>

tools/extra/test/clang-tidy/modernize-deprecated-headers-cxx11.cpp

-// RUN: %check_clang_tidy %s modernize-deprecated-headers %t -- -extra-arg-before=-isystem%S/Inputs/modernize-deprecated-headers -- -std=c++11 -v
+// RUN: %check_clang_tidy %s modernize-deprecated-headers %t -- -extra-arg-before=-isystem%S/Inputs/modernize-deprecated-headers -- -std=c++11 -v -nostdinc
 
 #include <assert.h>
 // CHECK-MESSAGES: :[[@LINE-1]]:10: warning: inclusion of deprecated C++ header 'assert.h'; consider using 'cassert' instead [modernize-deprecated-headers]
 // CHECK-FIXES: {{^}}#include <cassert>{{$}}
 #include <complex.h>
 // CHECK-MESSAGES: :[[@LINE-1]]:10: warning: inclusion of deprecated C++ header 'complex.h'; consider using 'complex' instead
 // CHECK-FIXES: {{^}}#include <complex>{{$}}
 #include <ctype.h>

tools/extra/test/clang-tidy/modernize-pass-by-value-macro-header.cpp

-// RUN: %check_clang_tidy %s modernize-pass-by-value %t -- -- -std=c++11 -isystem %S/Inputs/Headers
+// RUN: %check_clang_tidy %s modernize-pass-by-value %t -- -- -std=c++11 -isystem %S/Inputs/Headers -nostdinc
 
 // CHECK-FIXES: #include <utility>
 
 #define HEADER <./a.h>
 #include HEADER
 
 struct A {
   A(const A &) {}

tools/extra/test/clang-tidy/readability-identifier-naming.cpp

 // RUN:     {key: readability-identifier-naming.VirtualMethodCase, value: Camel_Snake_Case}, \
 // RUN:     {key: readability-identifier-naming.VirtualMethodPrefix, value: 'v_'}, \
 // RUN:     {key: readability-identifier-naming.MacroDefinitionCase, value: UPPER_CASE}, \
 // RUN:     {key: readability-identifier-naming.TypeAliasCase, value: camel_Snake_Back}, \
 // RUN:     {key: readability-identifier-naming.TypeAliasSuffix, value: '_t'}, \
 // RUN:     {key: readability-identifier-naming.IgnoreFailedSplit, value: 0} \
 // RUN:   ]}' -- -std=c++11 -fno-delayed-template-parsing \
 // RUN:   -I%S/Inputs/readability-identifier-naming \
-// RUN:   -isystem %S/Inputs/readability-identifier-naming/system
+// RUN:   -isystem %S/Inputs/readability-identifier-naming/system -nostdinc
 
 // clang-format off
 
 #include <system-header.h>
 #include "user-header.h"
 // NO warnings or fixes expected from declarations within header files without
 // the -header-filter= option