diff options
| author | Eli Friedman <eli.friedman@gmail.com> | 2012-12-18 02:37:32 +0000 |
|---|---|---|
| committer | Eli Friedman <eli.friedman@gmail.com> | 2012-12-18 02:37:32 +0000 |
| commit | 52f32b9b2be52b50377ec6a7315bbf079692aeaf (patch) | |
| tree | d1feb1254f6f960afb0e109d7bb3167f667d1063 /clang/lib | |
| parent | 62737b65b17349d08159c3126c16497d34a97ac1 (diff) | |
| download | bcm5719-llvm-52f32b9b2be52b50377ec6a7315bbf079692aeaf.tar.gz bcm5719-llvm-52f32b9b2be52b50377ec6a7315bbf079692aeaf.zip | |
The underlying type for an enum should be an integer type, not another enum.
(This change only affects ObjC.)
<rdar://problem/12857117>.
llvm-svn: 170402
Diffstat (limited to 'clang/lib')
| -rw-r--r-- | clang/lib/Sema/SemaDecl.cpp | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp index ab931a773fd..4bccc1c9870 100644 --- a/clang/lib/Sema/SemaDecl.cpp +++ b/clang/lib/Sema/SemaDecl.cpp @@ -8454,9 +8454,13 @@ bool Sema::CheckEnumUnderlyingType(TypeSourceInfo *TI) { SourceLocation UnderlyingLoc = TI->getTypeLoc().getBeginLoc(); QualType T = TI->getType(); - if (T->isDependentType() || T->isIntegralType(Context)) + if (T->isDependentType()) return false; + if (const BuiltinType *BT = T->getAs<BuiltinType>()) + if (BT->isInteger()) + return false; + Diag(UnderlyingLoc, diag::err_enum_invalid_underlying) << T; return true; } |
