Argyrios Kyrtzidis | def4e2a | 2011-01-31 07:05:00 +0000 | [diff] [blame] | 1 | // RUN: %clang_cc1 -fsyntax-only -Wnon-virtual-dtor -verify %s |
Douglas Gregor | 42a552f | 2008-11-05 20:51:48 +0000 | [diff] [blame] | 2 | class A { |
| 3 | public: |
| 4 | ~A(); |
| 5 | }; |
| 6 | |
| 7 | class B { |
| 8 | public: |
| 9 | ~B() { } |
| 10 | }; |
| 11 | |
| 12 | class C { |
| 13 | public: |
| 14 | (~C)() { } |
| 15 | }; |
| 16 | |
| 17 | struct D { |
| 18 | static void ~D(int, ...) const { } // \ |
| 19 | // expected-error{{type qualifier is not allowed on this function}} \ |
| 20 | // expected-error{{destructor cannot be declared 'static'}} \ |
Douglas Gregor | 42a552f | 2008-11-05 20:51:48 +0000 | [diff] [blame] | 21 | // expected-error{{destructor cannot have any parameters}} \ |
Douglas Gregor | d92ec47 | 2010-07-01 05:10:53 +0000 | [diff] [blame] | 22 | // expected-error{{destructor cannot be variadic}} \ |
| 23 | // expected-error{{destructor cannot have a return type}} \ |
| 24 | // expected-error{{'const' qualifier is not allowed on a destructor}} |
Douglas Gregor | 42a552f | 2008-11-05 20:51:48 +0000 | [diff] [blame] | 25 | }; |
| 26 | |
Chris Lattner | 6540180 | 2009-04-25 08:28:21 +0000 | [diff] [blame] | 27 | struct D2 { |
| 28 | void ~D2() { } // \ |
| 29 | // expected-error{{destructor cannot have a return type}} |
| 30 | }; |
| 31 | |
Chris Lattner | eaaebc7 | 2009-04-25 08:06:05 +0000 | [diff] [blame] | 32 | |
Douglas Gregor | 42a552f | 2008-11-05 20:51:48 +0000 | [diff] [blame] | 33 | struct E; |
| 34 | |
| 35 | typedef E E_typedef; |
| 36 | struct E { |
John McCall | 7c2342d | 2010-03-10 11:27:22 +0000 | [diff] [blame] | 37 | ~E_typedef(); // expected-error{{destructor cannot be declared using a typedef 'E_typedef' (aka 'E') of the class name}} |
Douglas Gregor | 42a552f | 2008-11-05 20:51:48 +0000 | [diff] [blame] | 38 | }; |
| 39 | |
| 40 | struct F { |
Chris Lattner | 5f4a682 | 2008-11-23 23:12:31 +0000 | [diff] [blame] | 41 | (~F)(); // expected-note {{previous declaration is here}} |
| 42 | ~F(); // expected-error {{destructor cannot be redeclared}} |
Douglas Gregor | 42a552f | 2008-11-05 20:51:48 +0000 | [diff] [blame] | 43 | }; |
| 44 | |
Douglas Gregor | 124b878 | 2010-02-16 19:09:40 +0000 | [diff] [blame] | 45 | ~; // expected-error {{expected a class name after '~' to name a destructor}} |
Fariborz Jahanian | 76ed9cb | 2009-07-20 22:41:12 +0000 | [diff] [blame] | 46 | ~undef(); // expected-error {{expected the class name after '~' to name a destructor}} |
Douglas Gregor | 124b878 | 2010-02-16 19:09:40 +0000 | [diff] [blame] | 47 | ~operator+(int, int); // expected-error {{expected a class name after '~' to name a destructor}} |
Argyrios Kyrtzidis | c7ed9c6 | 2008-11-07 22:02:30 +0000 | [diff] [blame] | 48 | ~F(){} // expected-error {{destructor must be a non-static member function}} |
Douglas Gregor | 9d35097 | 2008-12-12 08:25:50 +0000 | [diff] [blame] | 49 | |
| 50 | struct G { |
| 51 | ~G(); |
| 52 | }; |
| 53 | |
| 54 | G::~G() { } |
| 55 | |
Anders Carlsson | 7786d1c | 2009-04-30 23:18:11 +0000 | [diff] [blame] | 56 | // <rdar://problem/6841210> |
| 57 | struct H { |
| 58 | ~H(void) { } |
| 59 | }; |
Fariborz Jahanian | c19f959 | 2009-07-21 15:28:50 +0000 | [diff] [blame] | 60 | |
| 61 | struct X {}; |
| 62 | |
| 63 | struct Y { |
| 64 | ~X(); // expected-error {{expected the class name after '~' to name the enclosing class}} |
| 65 | }; |
Douglas Gregor | 333de06 | 2010-02-25 18:11:54 +0000 | [diff] [blame] | 66 | |
| 67 | namespace PR6421 { |
| 68 | class T; // expected-note{{forward declaration}} |
| 69 | |
Douglas Gregor | 200b292 | 2010-09-17 22:25:06 +0000 | [diff] [blame] | 70 | class QGenericArgument // expected-note{{declared here}} |
Douglas Gregor | 333de06 | 2010-02-25 18:11:54 +0000 | [diff] [blame] | 71 | { |
| 72 | template<typename U> |
| 73 | void foo(T t) // expected-error{{variable has incomplete type}} |
| 74 | { } |
| 75 | |
| 76 | void disconnect() |
| 77 | { |
| 78 | T* t; |
Douglas Gregor | 200b292 | 2010-09-17 22:25:06 +0000 | [diff] [blame] | 79 | bob<QGenericArgument>(t); // expected-error{{undeclared identifier 'bob'}} \ |
| 80 | // expected-error{{does not refer to a value}} |
Douglas Gregor | 333de06 | 2010-02-25 18:11:54 +0000 | [diff] [blame] | 81 | } |
| 82 | }; |
| 83 | } |
Douglas Gregor | 9c12739 | 2010-03-26 06:57:13 +0000 | [diff] [blame] | 84 | |
| 85 | namespace PR6709 { |
| 86 | template<class T> class X { T v; ~X() { ++*v; } }; |
| 87 | void a(X<int> x) {} |
| 88 | } |
Douglas Gregor | d92ec47 | 2010-07-01 05:10:53 +0000 | [diff] [blame] | 89 | |
| 90 | struct X0 { virtual ~X0() throw(); }; |
| 91 | struct X1 : public X0 { }; |
John McCall | 5efd91a | 2010-07-03 18:33:00 +0000 | [diff] [blame] | 92 | |
| 93 | // Make sure we instantiate operator deletes when building a virtual |
| 94 | // destructor. |
| 95 | namespace test6 { |
| 96 | template <class T> class A { |
| 97 | public: |
Benjamin Kramer | 6b85512 | 2010-07-03 20:33:13 +0000 | [diff] [blame] | 98 | void *operator new(__SIZE_TYPE__); |
John McCall | 5efd91a | 2010-07-03 18:33:00 +0000 | [diff] [blame] | 99 | void operator delete(void *p) { |
| 100 | T::deleteIt(p); // expected-error {{type 'int' cannot be used prior to '::'}} |
| 101 | } |
| 102 | |
Nick Lewycky | 2a5f99e | 2010-11-25 00:35:20 +0000 | [diff] [blame] | 103 | virtual ~A() {} |
John McCall | 5efd91a | 2010-07-03 18:33:00 +0000 | [diff] [blame] | 104 | }; |
| 105 | |
Nick Lewycky | 2a5f99e | 2010-11-25 00:35:20 +0000 | [diff] [blame] | 106 | class B : A<int> { B(); }; // expected-note {{in instantiation of member function 'test6::A<int>::operator delete' requested here}} |
| 107 | B::B() {} |
John McCall | 5efd91a | 2010-07-03 18:33:00 +0000 | [diff] [blame] | 108 | } |
John McCall | fcadea2 | 2010-08-12 00:57:17 +0000 | [diff] [blame] | 109 | |
| 110 | // Make sure classes are marked invalid when they have invalid |
| 111 | // members. This avoids a crash-on-invalid. |
| 112 | namespace test7 { |
| 113 | struct A { |
| 114 | ~A() const; // expected-error {{'const' qualifier is not allowed on a destructor}} |
| 115 | }; |
| 116 | struct B : A {}; |
| 117 | |
| 118 | void test() { |
| 119 | B *b; |
| 120 | b->~B(); |
| 121 | } |
| 122 | } |
Argyrios Kyrtzidis | def4e2a | 2011-01-31 07:05:00 +0000 | [diff] [blame] | 123 | |
| 124 | namespace nonvirtualdtor { |
| 125 | struct S1 { // expected-warning {{has virtual functions but non-virtual destructor}} |
| 126 | virtual void m(); |
| 127 | }; |
| 128 | |
| 129 | struct S2 { |
| 130 | ~S2(); // expected-warning {{has virtual functions but non-virtual destructor}} |
| 131 | virtual void m(); |
| 132 | }; |
| 133 | |
| 134 | struct S3 : public S1 { // expected-warning {{has virtual functions but non-virtual destructor}} |
| 135 | virtual void m(); |
| 136 | }; |
| 137 | |
| 138 | struct S4 : public S2 { // expected-warning {{has virtual functions but non-virtual destructor}} |
| 139 | virtual void m(); |
| 140 | }; |
| 141 | |
| 142 | struct B { |
| 143 | virtual ~B(); |
| 144 | virtual void m(); |
| 145 | }; |
| 146 | |
| 147 | struct S5 : public B { |
| 148 | virtual void m(); |
| 149 | }; |
Argyrios Kyrtzidis | 9641fc8 | 2011-01-31 17:10:25 +0000 | [diff] [blame] | 150 | |
| 151 | struct S6 { |
| 152 | virtual void m(); |
| 153 | private: |
| 154 | ~S6(); |
| 155 | }; |
| 156 | |
| 157 | struct S7 { |
| 158 | virtual void m(); |
| 159 | protected: |
| 160 | ~S7(); |
| 161 | }; |
Argyrios Kyrtzidis | 668fdd8 | 2011-02-02 18:47:41 +0000 | [diff] [blame] | 162 | |
| 163 | template<class T> class TS : public B { |
| 164 | virtual void m(); |
| 165 | }; |
| 166 | |
| 167 | TS<int> baz; |
| 168 | |
| 169 | template<class T> class TS2 { // expected-warning {{'nonvirtualdtor::TS2<int>' has virtual functions but non-virtual destructor}} |
| 170 | virtual void m(); |
| 171 | }; |
| 172 | |
| 173 | TS2<int> foo; // expected-note {{instantiation}} |
Argyrios Kyrtzidis | def4e2a | 2011-01-31 07:05:00 +0000 | [diff] [blame] | 174 | } |