diff options
author | Anders Carlsson <andersca@mac.com> | 2010-10-06 02:43:25 +0000 |
---|---|---|
committer | Anders Carlsson <andersca@mac.com> | 2010-10-06 02:43:25 +0000 |
commit | 175ffbf7db893181b301e201c561ac928266f4be (patch) | |
tree | c60cf2f431697f6ef2e5c7c94afab37bb936a8bf /lib | |
parent | 4a1cdb5803ea0ad49ccbefb7f9ae14df9d677173 (diff) |
When checking for uninitialized fields in member initializers, special case static variables and enums. Fixes PR8075.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@115732 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib')
-rw-r--r-- | lib/Sema/SemaDeclCXX.cpp | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/lib/Sema/SemaDeclCXX.cpp b/lib/Sema/SemaDeclCXX.cpp index 1e4065454e..cbb16fe17e 100644 --- a/lib/Sema/SemaDeclCXX.cpp +++ b/lib/Sema/SemaDeclCXX.cpp @@ -1163,6 +1163,19 @@ static bool InitExprContainsUninitializedFields(const Stmt *S, } if (const MemberExpr *ME = dyn_cast<MemberExpr>(S)) { const NamedDecl *RhsField = ME->getMemberDecl(); + + if (const VarDecl *VD = dyn_cast<VarDecl>(RhsField)) { + // The member expression points to a static data member. + assert(VD->isStaticDataMember() && + "Member points to non-static data member!"); + return false; + } + + if (isa<EnumConstantDecl>(RhsField)) { + // The member expression points to an enum. + return false; + } + if (RhsField == LhsField) { // Initializing a field with itself. Throw a warning. // But wait; there are exceptions! |