Daniel Dunbar | a572887 | 2009-12-15 20:14:24 +0000 | [diff] [blame] | 1 | // RUN: %clang_cc1 -Wreorder -fsyntax-only -verify %s |
Douglas Gregor | 7ad8390 | 2008-11-05 04:29:56 +0000 | [diff] [blame] | 2 | class A { |
| 3 | int m; |
John McCall | 7002f4c | 2010-04-09 19:03:51 +0000 | [diff] [blame] | 4 | public: |
Fariborz Jahanian | bcfad54 | 2009-06-30 23:26:25 +0000 | [diff] [blame] | 5 | A() : A::m(17) { } // expected-error {{member initializer 'm' does not name a non-static data member or base class}} |
Fariborz Jahanian | d7b27e1 | 2009-07-23 00:42:24 +0000 | [diff] [blame] | 6 | A(int); |
Douglas Gregor | 7ad8390 | 2008-11-05 04:29:56 +0000 | [diff] [blame] | 7 | }; |
| 8 | |
| 9 | class B : public A { |
| 10 | public: |
| 11 | B() : A(), m(1), n(3.14) { } |
| 12 | |
| 13 | private: |
| 14 | int m; |
| 15 | float n; |
| 16 | }; |
| 17 | |
| 18 | |
| 19 | class C : public virtual B { |
| 20 | public: |
| 21 | C() : B() { } |
| 22 | }; |
| 23 | |
| 24 | class D : public C { |
| 25 | public: |
| 26 | D() : B(), C() { } |
| 27 | }; |
| 28 | |
| 29 | class E : public D, public B { |
| 30 | public: |
John McCall | 7c2342d | 2010-03-10 11:27:22 +0000 | [diff] [blame] | 31 | E() : B(), D() { } // expected-error{{base class initializer 'B' names both a direct base class and an inherited virtual base class}} |
Douglas Gregor | 7ad8390 | 2008-11-05 04:29:56 +0000 | [diff] [blame] | 32 | }; |
| 33 | |
| 34 | |
| 35 | typedef int INT; |
| 36 | |
| 37 | class F : public B { |
| 38 | public: |
| 39 | int B; |
| 40 | |
| 41 | F() : B(17), |
| 42 | m(17), // expected-error{{member initializer 'm' does not name a non-static data member or base class}} |
Chris Lattner | d0344a4 | 2009-02-19 23:45:49 +0000 | [diff] [blame] | 43 | INT(17) // expected-error{{constructor initializer 'INT' (aka 'int') does not name a class}} |
Douglas Gregor | 7ad8390 | 2008-11-05 04:29:56 +0000 | [diff] [blame] | 44 | { |
| 45 | } |
| 46 | }; |
Douglas Gregor | 3f08d18 | 2008-11-10 16:59:40 +0000 | [diff] [blame] | 47 | |
| 48 | class G : A { |
| 49 | G() : A(10); // expected-error{{expected '{'}} |
| 50 | }; |
Anders Carlsson | a7b3521 | 2009-03-25 02:58:17 +0000 | [diff] [blame] | 51 | |
| 52 | void f() : a(242) { } // expected-error{{only constructors take base initializers}} |
| 53 | |
| 54 | class H : A { |
| 55 | H(); |
| 56 | }; |
| 57 | |
| 58 | H::H() : A(10) { } |
| 59 | |
Fariborz Jahanian | 9da7201 | 2009-06-30 17:34:52 +0000 | [diff] [blame] | 60 | |
| 61 | class X {}; |
| 62 | class Y {}; |
| 63 | |
| 64 | struct S : Y, virtual X { |
| 65 | S (); |
| 66 | }; |
| 67 | |
| 68 | struct Z : S { |
John McCall | 7c2342d | 2010-03-10 11:27:22 +0000 | [diff] [blame] | 69 | Z() : X(), S(), E() {} // expected-error {{type 'E' is not a direct or virtual base of 'Z'}} |
Fariborz Jahanian | 9da7201 | 2009-06-30 17:34:52 +0000 | [diff] [blame] | 70 | }; |
| 71 | |
Fariborz Jahanian | 5ac3dfc | 2009-06-30 21:52:59 +0000 | [diff] [blame] | 72 | class U { |
| 73 | union { int a; char* p; }; |
| 74 | union { int b; double d; }; |
| 75 | |
John McCall | 3c3ccdb | 2010-04-10 09:28:51 +0000 | [diff] [blame] | 76 | U() : a(1), // expected-note {{previous initialization is here}} |
| 77 | p(0), // expected-error {{initializing multiple members of anonymous union}} |
| 78 | d(1.0) {} |
Fariborz Jahanian | 5ac3dfc | 2009-06-30 21:52:59 +0000 | [diff] [blame] | 79 | }; |
| 80 | |
Fariborz Jahanian | bcfad54 | 2009-06-30 23:26:25 +0000 | [diff] [blame] | 81 | struct V {}; |
| 82 | struct Base {}; |
| 83 | struct Base1 {}; |
| 84 | |
| 85 | struct Derived : Base, Base1, virtual V { |
| 86 | Derived (); |
| 87 | }; |
| 88 | |
| 89 | struct Current : Derived { |
| 90 | int Derived; |
John McCall | d6ca8da | 2010-04-10 07:37:23 +0000 | [diff] [blame] | 91 | Current() : Derived(1), ::Derived(), // expected-warning {{field 'Derived' will be initialized after base '::Derived'}} \ |
| 92 | // expected-warning {{base class '::Derived' will be initialized after base 'Derived::V'}} |
Fariborz Jahanian | bcfad54 | 2009-06-30 23:26:25 +0000 | [diff] [blame] | 93 | ::Derived::Base(), // expected-error {{type '::Derived::Base' is not a direct or virtual base of 'Current'}} |
| 94 | Derived::Base1(), // expected-error {{type 'Derived::Base1' is not a direct or virtual base of 'Current'}} |
John McCall | d6ca8da | 2010-04-10 07:37:23 +0000 | [diff] [blame] | 95 | Derived::V(), |
Fariborz Jahanian | bcfad54 | 2009-06-30 23:26:25 +0000 | [diff] [blame] | 96 | ::NonExisting(), // expected-error {{member initializer 'NonExisting' does not name a non-static data member or}} |
| 97 | INT::NonExisting() {} // expected-error {{expected a class or namespace}} \ |
Mike Stump | 1eb4433 | 2009-09-09 15:08:12 +0000 | [diff] [blame] | 98 | // expected-error {{member initializer 'NonExisting' does not name a non-static data member or}} |
Fariborz Jahanian | bcfad54 | 2009-06-30 23:26:25 +0000 | [diff] [blame] | 99 | }; |
Fariborz Jahanian | 87595e4 | 2009-07-23 23:32:59 +0000 | [diff] [blame] | 100 | |
John McCall | 220ccbf | 2010-01-13 00:25:19 +0000 | [diff] [blame] | 101 | struct M { // expected-note 2 {{candidate constructor (the implicit copy constructor)}} \ |
Eli Friedman | 49c16da | 2009-11-09 01:05:47 +0000 | [diff] [blame] | 102 | // expected-note {{declared here}} \ |
| 103 | // expected-note {{declared here}} |
John McCall | b1622a1 | 2010-01-06 09:43:14 +0000 | [diff] [blame] | 104 | M(int i, int j); // expected-note 2 {{candidate constructor}} |
Fariborz Jahanian | 87595e4 | 2009-07-23 23:32:59 +0000 | [diff] [blame] | 105 | }; |
| 106 | |
| 107 | struct N : M { |
John McCall | 7c2342d | 2010-03-10 11:27:22 +0000 | [diff] [blame] | 108 | N() : M(1), // expected-error {{no matching constructor for initialization of 'M'}} |
| 109 | m1(100) { } // expected-error {{no matching constructor for initialization of 'M'}} |
Fariborz Jahanian | 87595e4 | 2009-07-23 23:32:59 +0000 | [diff] [blame] | 110 | M m1; |
| 111 | }; |
| 112 | |
Eli Friedman | 49c16da | 2009-11-09 01:05:47 +0000 | [diff] [blame] | 113 | struct P : M { |
John McCall | 7c2342d | 2010-03-10 11:27:22 +0000 | [diff] [blame] | 114 | P() { } // expected-error {{constructor for 'P' must explicitly initialize the base class 'M' which does not have a default constructor}} \ |
Eli Friedman | 49c16da | 2009-11-09 01:05:47 +0000 | [diff] [blame] | 115 | // expected-error {{member 'm'}} |
| 116 | M m; // expected-note {{member is declared here}} |
Fariborz Jahanian | 87595e4 | 2009-07-23 23:32:59 +0000 | [diff] [blame] | 117 | }; |
| 118 | |
Fariborz Jahanian | 7252f51 | 2009-07-24 20:28:49 +0000 | [diff] [blame] | 119 | struct Q { |
Douglas Gregor | 9db7dbb | 2010-01-31 09:12:51 +0000 | [diff] [blame] | 120 | Q() : f1(1,2), // expected-error {{excess elements in scalar initializer}} |
| 121 | pf(0.0) { } // expected-error {{cannot initialize a member subobject of type 'float *' with an rvalue of type 'double'}} |
Fariborz Jahanian | 7252f51 | 2009-07-24 20:28:49 +0000 | [diff] [blame] | 122 | float f1; |
| 123 | |
| 124 | float *pf; |
| 125 | }; |
John McCall | b419004 | 2009-11-04 23:02:40 +0000 | [diff] [blame] | 126 | |
| 127 | // A silly class used to demonstrate field-is-uninitialized in constructors with |
| 128 | // multiple params. |
John McCall | 7002f4c | 2010-04-09 19:03:51 +0000 | [diff] [blame] | 129 | class TwoInOne { public: TwoInOne(TwoInOne a, TwoInOne b) {} }; |
John McCall | b419004 | 2009-11-04 23:02:40 +0000 | [diff] [blame] | 130 | class InitializeUsingSelfTest { |
| 131 | bool A; |
| 132 | char* B; |
| 133 | int C; |
| 134 | TwoInOne D; |
| 135 | InitializeUsingSelfTest(int E) |
| 136 | : A(A), // expected-warning {{field is uninitialized when used here}} |
| 137 | B((((B)))), // expected-warning {{field is uninitialized when used here}} |
| 138 | C(A && InitializeUsingSelfTest::C), // expected-warning {{field is uninitialized when used here}} |
| 139 | D(D, // expected-warning {{field is uninitialized when used here}} |
| 140 | D) {} // expected-warning {{field is uninitialized when used here}} |
| 141 | }; |
| 142 | |
| 143 | int IntWrapper(int i) { return 0; }; |
| 144 | class InitializeUsingSelfExceptions { |
| 145 | int A; |
| 146 | int B; |
Argyrios Kyrtzidis | ff8819b | 2010-09-21 10:47:20 +0000 | [diff] [blame] | 147 | int C; |
| 148 | void *P; |
John McCall | b419004 | 2009-11-04 23:02:40 +0000 | [diff] [blame] | 149 | InitializeUsingSelfExceptions(int B) |
| 150 | : A(IntWrapper(A)), // Due to a conservative implementation, we do not report warnings inside function/ctor calls even though it is possible to do so. |
Argyrios Kyrtzidis | ff8819b | 2010-09-21 10:47:20 +0000 | [diff] [blame] | 151 | B(B), // Not a warning; B is a local variable. |
| 152 | C(sizeof(C)), // sizeof doesn't reference contents, do not warn |
| 153 | P(&P) {} // address-of doesn't reference contents (the pointer may be dereferenced in the same expression but it would be rare; and weird) |
John McCall | b419004 | 2009-11-04 23:02:40 +0000 | [diff] [blame] | 154 | }; |
| 155 | |
| 156 | class CopyConstructorTest { |
| 157 | bool A, B, C; |
| 158 | CopyConstructorTest(const CopyConstructorTest& rhs) |
| 159 | : A(rhs.A), |
| 160 | B(B), // expected-warning {{field is uninitialized when used here}} |
| 161 | C(rhs.C || C) { } // expected-warning {{field is uninitialized when used here}} |
| 162 | }; |
Douglas Gregor | c07a494 | 2009-11-15 08:51:10 +0000 | [diff] [blame] | 163 | |
| 164 | // Make sure we aren't marking default constructors when we shouldn't be. |
| 165 | template<typename T> |
| 166 | struct NDC { |
| 167 | T &ref; |
| 168 | |
| 169 | NDC() { } |
| 170 | NDC(T &ref) : ref(ref) { } |
| 171 | }; |
| 172 | |
| 173 | struct X0 : NDC<int> { |
| 174 | X0(int &ref) : NDC<int>(ref), ndc(ref) { } |
| 175 | |
| 176 | NDC<int> ndc; |
| 177 | }; |
Anders Carlsson | 114a297 | 2010-04-23 03:07:47 +0000 | [diff] [blame] | 178 | |
| 179 | namespace Test0 { |
| 180 | |
| 181 | struct A { A(); }; |
| 182 | |
| 183 | struct B { |
| 184 | B() { } |
| 185 | const A a; |
| 186 | }; |
| 187 | |
| 188 | } |
John McCall | 0d6b164 | 2010-04-23 18:46:30 +0000 | [diff] [blame] | 189 | |
Anders Carlsson | 9fd682b | 2010-04-25 01:00:05 +0000 | [diff] [blame] | 190 | namespace Test1 { |
John McCall | 0d6b164 | 2010-04-23 18:46:30 +0000 | [diff] [blame] | 191 | struct A { |
| 192 | enum Kind { Foo } Kind; |
| 193 | A() : Kind(Foo) {} |
| 194 | }; |
| 195 | } |
Anders Carlsson | 9fd682b | 2010-04-25 01:00:05 +0000 | [diff] [blame] | 196 | |
| 197 | namespace Test2 { |
| 198 | |
| 199 | struct A { |
| 200 | A(const A&); |
| 201 | }; |
| 202 | |
| 203 | struct B : virtual A { }; |
| 204 | struct C : A, B { }; |
| 205 | |
| 206 | C f(C c) { |
| 207 | return c; |
| 208 | } |
| 209 | |
| 210 | } |
Chandler Carruth | e861c60 | 2010-06-30 02:59:29 +0000 | [diff] [blame] | 211 | |
| 212 | // Don't build implicit initializers for anonymous union fields when we already |
| 213 | // have an explicit initializer for another field in the union. |
| 214 | namespace PR7402 { |
| 215 | struct S { |
| 216 | union { |
| 217 | void* ptr_; |
| 218 | struct { int i_; }; |
| 219 | }; |
| 220 | |
| 221 | template <typename T> S(T) : ptr_(0) { } |
| 222 | }; |
| 223 | |
| 224 | void f() { |
| 225 | S s(3); |
| 226 | } |
| 227 | } |
John McCall | 572fc62 | 2010-08-17 07:23:57 +0000 | [diff] [blame] | 228 | |
| 229 | // <rdar://problem/8308215>: don't crash. |
| 230 | // Lots of questionable recovery here; errors can change. |
| 231 | namespace test3 { |
| 232 | class A : public std::exception {}; // expected-error {{undeclared identifier}} expected-error {{expected class name}} expected-note 3 {{candidate}} expected-note {{passing argument}} |
| 233 | class B : public A { |
| 234 | public: |
| 235 | B(const String& s, int e=0) // expected-error {{unknown type name}} |
| 236 | : A(e), m_String(s) , m_ErrorStr(__null) {} // expected-error {{no matching constructor}} expected-error {{does not name}} |
| 237 | B(const B& e) |
| 238 | : A(e), m_String(e.m_String), m_ErrorStr(__null) { // expected-error {{no viable conversion}} expected-error {{does not name}} |
| 239 | } |
| 240 | }; |
| 241 | } |
Anders Carlsson | 175ffbf | 2010-10-06 02:43:25 +0000 | [diff] [blame] | 242 | |
| 243 | // PR8075 |
| 244 | namespace PR8075 { |
| 245 | |
| 246 | struct S1 { |
| 247 | enum { FOO = 42 }; |
| 248 | static const int bar = 42; |
| 249 | static int baz(); |
| 250 | S1(int); |
| 251 | }; |
| 252 | |
| 253 | const int S1::bar; |
| 254 | |
| 255 | struct S2 { |
| 256 | S1 s1; |
| 257 | S2() : s1(s1.FOO) {} |
| 258 | }; |
| 259 | |
| 260 | struct S3 { |
| 261 | S1 s1; |
| 262 | S3() : s1(s1.bar) {} |
| 263 | }; |
| 264 | |
| 265 | struct S4 { |
| 266 | S1 s1; |
| 267 | S4() : s1(s1.baz()) {} |
| 268 | }; |
| 269 | |
| 270 | } |