maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Sema::ActOnEnumBody(): handle nested enum redefinitions. 

Fixes <rdar://problem/6093889> Nested enum redefinition crashes sema.

llvm-svn: 54463
This commit is contained in:
Steve Naroff 2008-08-07 14:08:16 +00:00
parent 62a3c3c11b
commit f960c0bb9e
1 changed files with 13 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
clang/lib/Sema/SemaDecl.cpp
View File

@ -2223,11 +2223,23 @@ Sema::DeclTy *Sema::ActOnEnumConstant(Scope *S, DeclTy *theEnumDecl,
return New;
}
// FIXME: For consistency with ActOnFields(), we should have the parser
// pass in the source location for the left/right braces.
void Sema::ActOnEnumBody(SourceLocation EnumLoc, DeclTy *EnumDeclX,
DeclTy **Elements, unsigned NumElements) {
EnumDecl *Enum = cast<EnumDecl>(static_cast<Decl*>(EnumDeclX));
assert(!Enum->isDefinition() && "Enum redefinitions can't reach here");
if (Enum && Enum->isDefinition()) {
// Diagnose code like:
// enum e0 {
// E0 = sizeof(enum e0 { E1 })
// };
Diag(Enum->getLocation(), diag::err_nested_redefinition,
Enum->getName());
Diag(EnumLoc, diag::err_previous_definition);
Enum->setInvalidDecl();
return;
}
// TODO: If the result value doesn't fit in an int, it must be a long or long
// long value. ISO C does not support this, but GCC does as an extension,
// emit a warning.