diff options
author | Ted Kremenek <kremenek@apple.com> | 2012-05-09 05:35:08 +0000 |
---|---|---|
committer | Ted Kremenek <kremenek@apple.com> | 2012-05-09 05:35:08 +0000 |
commit | 00e1f6f5fed4f33f67181578be0232accd342282 (patch) | |
tree | 95cebec5ab347f9838b1d10cf3f60f2250102048 /lib/Sema/SemaChecking.cpp | |
parent | 4493c0aa8add05b2fc534b0d8970c4b5480fad3b (diff) |
Teach IsTailPaddedMemberArray() (used by -Warray-bounds) that a FieldDecl may have a Typedef type, and not always a ConstantArrayType.
Fixes <rdar://problem/11387038>.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@156464 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Sema/SemaChecking.cpp')
-rw-r--r-- | lib/Sema/SemaChecking.cpp | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/lib/Sema/SemaChecking.cpp b/lib/Sema/SemaChecking.cpp index d538fcf36d..bc4452f125 100644 --- a/lib/Sema/SemaChecking.cpp +++ b/lib/Sema/SemaChecking.cpp @@ -4590,12 +4590,20 @@ static bool IsTailPaddedMemberArray(Sema &S, llvm::APInt Size, // substitution to form C89 tail-padded arrays. TypeSourceInfo *TInfo = FD->getTypeSourceInfo(); - if (TInfo) { - ConstantArrayTypeLoc TL = - cast<ConstantArrayTypeLoc>(TInfo->getTypeLoc()); - const Expr *SizeExpr = dyn_cast<IntegerLiteral>(TL.getSizeExpr()); + while (TInfo) { + TypeLoc TL = TInfo->getTypeLoc(); + // Look through typedefs. + const TypedefTypeLoc *TTL = dyn_cast<TypedefTypeLoc>(&TL); + if (TTL) { + const TypedefNameDecl *TDL = TTL->getTypedefNameDecl(); + TInfo = TDL->getTypeSourceInfo(); + continue; + } + ConstantArrayTypeLoc CTL = cast<ConstantArrayTypeLoc>(TL); + const Expr *SizeExpr = dyn_cast<IntegerLiteral>(CTL.getSizeExpr()); if (!SizeExpr || SizeExpr->getExprLoc().isMacroID()) return false; + break; } const RecordDecl *RD = dyn_cast<RecordDecl>(FD->getDeclContext()); |