diff options
| author | David Blaikie <dblaikie@gmail.com> | 2011-11-17 06:01:57 +0000 |
|---|---|---|
| committer | David Blaikie <dblaikie@gmail.com> | 2011-11-17 06:01:57 +0000 |
| commit | 0f65d59e5224ad9ed16ae0db9c700b73b7988d94 (patch) | |
| tree | c1414763c5681ddd44be884811faabc03415f69e /clang/lib/Sema | |
| parent | e24c4acf6cd5ecbfed993c3b598dbc2fe24ab7ab (diff) | |
| download | bcm5719-llvm-0f65d59e5224ad9ed16ae0db9c700b73b7988d94.tar.gz bcm5719-llvm-0f65d59e5224ad9ed16ae0db9c700b73b7988d94.zip | |
Include named unions in union member init checking
llvm-svn: 144883
Diffstat (limited to 'clang/lib/Sema')
| -rw-r--r-- | clang/lib/Sema/SemaDeclCXX.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/clang/lib/Sema/SemaDeclCXX.cpp b/clang/lib/Sema/SemaDeclCXX.cpp index 8aeb8fb33f4..52eb9b0d3f3 100644 --- a/clang/lib/Sema/SemaDeclCXX.cpp +++ b/clang/lib/Sema/SemaDeclCXX.cpp @@ -3065,11 +3065,9 @@ bool CheckRedundantUnionInit(Sema &S, RedundantUnionMap &Unions) { FieldDecl *Field = Init->getAnyMember(); RecordDecl *Parent = Field->getParent(); - if (!Parent->isAnonymousStructOrUnion()) - return false; - NamedDecl *Child = Field; - do { + + while (Parent->isAnonymousStructOrUnion() || Parent->isUnion()) { if (Parent->isUnion()) { UnionEntry &En = Unions[Parent]; if (En.first && En.first != Child) { @@ -3085,11 +3083,13 @@ bool CheckRedundantUnionInit(Sema &S, En.first = Child; En.second = Init; } + if (!Parent->isAnonymousStructOrUnion()) + return false; } Child = Parent; Parent = cast<RecordDecl>(Parent->getDeclContext()); - } while (Parent->isAnonymousStructOrUnion()); + } return false; } |

