Make this fix-it test case actually fail when there is a problem; add

a test for access declarations and remove a (broken) test for removal
of default arguments.


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

1 files changed, 9 insertions, 8 deletions

diff --git a/test/FixIt/fixit.cpp b/test/FixIt/fixit.cpp
index 04b99c9416..ee93755775 100644
--- a/test/FixIt/fixit.cpp
+++ b/test/FixIt/fixit.cpp
@@ -1,4 +1,4 @@
-// RUN: %clang_cc1 -pedantic -fixit %s -o - | %clang_cc1 -fsyntax-only -pedantic -Werror -x c++ -
+// RUN: %clang_cc1 -pedantic -Wall -fixit %s -o - | %clang_cc1 -fsyntax-only -pedantic -Wall -Werror -x c++ -
 
 /* This is a test of the various code modification hints that are
    provided as part of warning or extension diagnostics. All of the
@@ -28,11 +28,12 @@ struct CT<0> { }; // expected-error{{'template<>'}}
 
 template<> class CT<1> { }; // expected-error{{tag type}}
 
-// PR5444
-namespace PR5444 {
-  void foo(int x, int y = 0);
-  void foo(int x, int y = 0) { }
+// Access declarations
+class A {
+protected:
+  int foo();
+};
 
-  void foo(int  = 0);
-  void foo(int  = 0) { }
-}
+class B : public A {
+  A::foo; // expected-warning{{access declarations are deprecated}}
+};