Index: cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td =================================================================== --- cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td +++ cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td @@ -4564,10 +4564,15 @@ "%select{typedef|type alias|type alias template}0 " "redefinition with different types%diff{ ($ vs $)|}1,2">; def err_tag_reference_non_tag : Error< - "elaborated type refers to %select{a non-tag type|a typedef|a type alias|a template|a type alias template|a template template argument}0">; + "%select{non-struct type|non-class type|non-union type|non-enum " + "type|typedef|type alias|template|type alias template|template " + "template argument}1 %0 cannot be referenced with a " + "%select{struct|interface|union|class|enum}2 specifier">; def err_tag_reference_conflict : Error< - "implicit declaration introduced by elaborated type conflicts with " - "%select{a declaration|a typedef|a type alias|a template}0 of the same name">; + "implicit declaration introduced by elaborated type conflicts with a " + "%select{non-struct type|non-class type|non-union type|non-enum " + "type|typedef|type alias|template|type alias template|template " + "template argument}0 of the same name">; def err_dependent_tag_decl : Error< "%select{declaration|definition}0 of " "%select{struct|interface|union|class|enum}1 in a dependent scope">; Index: cfe/trunk/include/clang/Sema/Sema.h =================================================================== --- cfe/trunk/include/clang/Sema/Sema.h +++ cfe/trunk/include/clang/Sema/Sema.h @@ -1976,7 +1976,10 @@ /// Common ways to introduce type names without a tag for use in diagnostics. /// Keep in sync with err_tag_reference_non_tag. enum NonTagKind { - NTK_Unknown, + NTK_NonStruct, + NTK_NonClass, + NTK_NonUnion, + NTK_NonEnum, NTK_Typedef, NTK_TypeAlias, NTK_Template, @@ -1986,7 +1989,7 @@ /// Given a non-tag type declaration, returns an enum useful for indicating /// what kind of non-tag type this is. - NonTagKind getNonTagTypeDeclKind(const Decl *D); + NonTagKind getNonTagTypeDeclKind(const Decl *D, TagTypeKind TTK); bool isAcceptableTagRedeclaration(const TagDecl *Previous, TagTypeKind NewTag, bool isDefinition, Index: cfe/trunk/lib/Sema/SemaDecl.cpp =================================================================== --- cfe/trunk/lib/Sema/SemaDecl.cpp +++ cfe/trunk/lib/Sema/SemaDecl.cpp @@ -12430,7 +12430,8 @@ return Tag == TTK_Struct || Tag == TTK_Class || Tag == TTK_Interface; } -Sema::NonTagKind Sema::getNonTagTypeDeclKind(const Decl *PrevDecl) { +Sema::NonTagKind Sema::getNonTagTypeDeclKind(const Decl *PrevDecl, + TagTypeKind TTK) { if (isa(PrevDecl)) return NTK_Typedef; else if (isa(PrevDecl)) @@ -12441,7 +12442,17 @@ return NTK_TypeAliasTemplate; else if (isa(PrevDecl)) return NTK_TemplateTemplateArgument; - return NTK_Unknown; + switch (TTK) { + case TTK_Struct: + case TTK_Interface: + case TTK_Class: + return getLangOpts().CPlusPlus ? NTK_NonClass : NTK_NonStruct; + case TTK_Union: + return NTK_NonUnion; + case TTK_Enum: + return NTK_NonEnum; + } + llvm_unreachable("invalid TTK"); } /// \brief Determine whether a tag with a given kind is acceptable @@ -13224,8 +13235,9 @@ // (non-redeclaration) lookup. if ((TUK == TUK_Reference || TUK == TUK_Friend) && !Previous.isForRedeclaration()) { - NonTagKind NTK = getNonTagTypeDeclKind(PrevDecl); - Diag(NameLoc, diag::err_tag_reference_non_tag) << NTK; + NonTagKind NTK = getNonTagTypeDeclKind(PrevDecl, Kind); + Diag(NameLoc, diag::err_tag_reference_non_tag) << PrevDecl << NTK + << Kind; Diag(PrevDecl->getLocation(), diag::note_declared_at); Invalid = true; @@ -13236,7 +13248,7 @@ // Diagnose implicit declarations introduced by elaborated types. } else if (TUK == TUK_Reference || TUK == TUK_Friend) { - NonTagKind NTK = getNonTagTypeDeclKind(PrevDecl); + NonTagKind NTK = getNonTagTypeDeclKind(PrevDecl, Kind); Diag(NameLoc, diag::err_tag_reference_conflict) << NTK; Diag(PrevDecl->getLocation(), diag::note_previous_decl) << PrevDecl; Invalid = true; Index: cfe/trunk/lib/Sema/SemaTemplate.cpp =================================================================== --- cfe/trunk/lib/Sema/SemaTemplate.cpp +++ cfe/trunk/lib/Sema/SemaTemplate.cpp @@ -2489,7 +2489,8 @@ // If the identifier resolves to a typedef-name or the simple-template-id // resolves to an alias template specialization, the // elaborated-type-specifier is ill-formed. - Diag(TemplateLoc, diag::err_tag_reference_non_tag) << NTK_TypeAliasTemplate; + Diag(TemplateLoc, diag::err_tag_reference_non_tag) + << TAT << NTK_TypeAliasTemplate << TagKind; Diag(TAT->getLocation(), diag::note_declared_at); } @@ -7508,8 +7509,8 @@ ClassTemplateDecl *ClassTemplate = dyn_cast(TD); if (!ClassTemplate) { - NonTagKind NTK = getNonTagTypeDeclKind(TD); - Diag(TemplateNameLoc, diag::err_tag_reference_non_tag) << NTK; + NonTagKind NTK = getNonTagTypeDeclKind(TD, Kind); + Diag(TemplateNameLoc, diag::err_tag_reference_non_tag) << TD << NTK << Kind; Diag(TD->getLocation(), diag::note_previous_use); return true; } Index: cfe/trunk/lib/Sema/TreeTransform.h =================================================================== --- cfe/trunk/lib/Sema/TreeTransform.h +++ cfe/trunk/lib/Sema/TreeTransform.h @@ -1013,8 +1013,9 @@ case LookupResult::FoundOverloaded: case LookupResult::FoundUnresolvedValue: { NamedDecl *SomeDecl = Result.getRepresentativeDecl(); - Sema::NonTagKind NTK = SemaRef.getNonTagTypeDeclKind(SomeDecl); - SemaRef.Diag(IdLoc, diag::err_tag_reference_non_tag) << NTK; + Sema::NonTagKind NTK = SemaRef.getNonTagTypeDeclKind(SomeDecl, Kind); + SemaRef.Diag(IdLoc, diag::err_tag_reference_non_tag) << SomeDecl + << NTK << Kind; SemaRef.Diag(SomeDecl->getLocation(), diag::note_declared_at); break; } @@ -5706,7 +5707,8 @@ Template.getAsTemplateDecl())) { SemaRef.Diag(TL.getNamedTypeLoc().getBeginLoc(), diag::err_tag_reference_non_tag) - << Sema::NTK_TypeAliasTemplate; + << TAT << Sema::NTK_TypeAliasTemplate + << ElaboratedType::getTagTypeKindForKeyword(T->getKeyword()); SemaRef.Diag(TAT->getLocation(), diag::note_declared_at); } } Index: cfe/trunk/test/CXX/basic/basic.lookup/basic.lookup.elab/p2.cpp =================================================================== --- cfe/trunk/test/CXX/basic/basic.lookup/basic.lookup.elab/p2.cpp +++ cfe/trunk/test/CXX/basic/basic.lookup/basic.lookup.elab/p2.cpp @@ -9,7 +9,7 @@ typedef int A; // expected-note {{declared here}} int test() { - struct A a; // expected-error {{elaborated type refers to a typedef}} + struct A a; // expected-error {{typedef 'A' cannot be referenced with a struct specifier}} return a.foo; } } @@ -18,7 +18,7 @@ template class A; // expected-note {{declared here}} int test() { - struct A a; // expected-error {{elaborated type refers to a template}} + struct A a; // expected-error {{template 'A' cannot be referenced with a struct specifier}} return a.foo; } } Index: cfe/trunk/test/CXX/dcl.dcl/dcl.spec/dcl.type/dcl.type.elab/p2-0x.cpp =================================================================== --- cfe/trunk/test/CXX/dcl.dcl/dcl.spec/dcl.type/dcl.type.elab/p2-0x.cpp +++ cfe/trunk/test/CXX/dcl.dcl/dcl.spec/dcl.type/dcl.type.elab/p2-0x.cpp @@ -2,18 +2,18 @@ struct A { typedef int type; }; template using X = A; // expected-note {{declared here}} -struct X* p2; // expected-error {{elaborated type refers to a type alias template}} +struct X* p2; // expected-error {{type alias template 'X' cannot be referenced with a struct specifier}} template using Id = T; // expected-note {{declared here}} template class F> struct Y { - struct F i; // expected-error {{elaborated type refers to a type alias template}} + struct F i; // expected-error {{type alias template 'Id' cannot be referenced with a struct specifier}} typename F::type j; // ok // FIXME: don't produce the diagnostic both for the definition and the instantiation. template using U = F; // expected-note 2{{declared here}} - struct Y::template U k; // expected-error 2{{elaborated type refers to a type alias template}} + struct Y::template U k; // expected-error 2{{type alias template 'U' cannot be referenced with a struct specifier}} typename Y::template U l; // ok }; template struct Y; // expected-note {{requested here}} Index: cfe/trunk/test/CXX/drs/dr2xx.cpp =================================================================== --- cfe/trunk/test/CXX/drs/dr2xx.cpp +++ cfe/trunk/test/CXX/drs/dr2xx.cpp @@ -620,7 +620,7 @@ template struct A { typedef typename T::type type; // ok even if this is a typedef-name, because // it's not an elaborated-type-specifier - typedef struct T::type foo; // expected-error {{elaborated type refers to a typedef}} + typedef struct T::type foo; // expected-error {{typedef 'type' cannot be referenced with a struct specifier}} }; struct B { struct type {}; }; struct C { typedef struct {} type; }; // expected-note {{here}} @@ -1048,8 +1048,8 @@ C::type i3; struct A a; - struct B b; // expected-error {{refers to a typedef}} - struct C c; // expected-error {{refers to a typedef}} + struct B b; // expected-error {{typedef 'B' cannot be referenced with a struct specifier}} + struct C c; // expected-error {{typedef 'C' cannot be referenced with a struct specifier}} B::B() {} // expected-error {{requires a type specifier}} B::A() {} // ok Index: cfe/trunk/test/CXX/drs/dr4xx.cpp =================================================================== --- cfe/trunk/test/CXX/drs/dr4xx.cpp +++ cfe/trunk/test/CXX/drs/dr4xx.cpp @@ -90,7 +90,7 @@ struct S *p; { typedef struct S S; // expected-note {{here}} - struct S *p; // expected-error {{refers to a typedef}} + struct S *p; // expected-error {{typedef 'S' cannot be referenced with a struct specifier}} } } struct S {}; Index: cfe/trunk/test/CXX/temp/temp.decls/temp.friend/p1.cpp =================================================================== --- cfe/trunk/test/CXX/temp/temp.decls/temp.friend/p1.cpp +++ cfe/trunk/test/CXX/temp/temp.decls/temp.friend/p1.cpp @@ -174,7 +174,7 @@ // This shouldn't crash. template class D { - friend class A; // expected-error {{elaborated type refers to a template}} + friend class A; // expected-error {{template 'A' cannot be referenced with a class specifier}} }; template class D; } Index: cfe/trunk/test/CXX/temp/temp.spec/no-body.cpp =================================================================== --- cfe/trunk/test/CXX/temp/temp.spec/no-body.cpp +++ cfe/trunk/test/CXX/temp/temp.spec/no-body.cpp @@ -43,7 +43,7 @@ namespace unsupported { #ifndef FIXING - template struct y; // expected-error {{elaborated type refers to a template}} + template struct y; // expected-error {{template 'y' cannot be referenced with a struct specifier}} #endif } Index: cfe/trunk/test/SemaCXX/PR8755.cpp =================================================================== --- cfe/trunk/test/SemaCXX/PR8755.cpp +++ cfe/trunk/test/SemaCXX/PR8755.cpp @@ -7,7 +7,7 @@ template void f() { - class A ::iterator foo; // expected-error{{elaborated type refers to a typedef}} + class A ::iterator foo; // expected-error{{typedef 'iterator' cannot be referenced with a class specifier}} } void g() { Index: cfe/trunk/test/SemaCXX/using-decl-templates.cpp =================================================================== --- cfe/trunk/test/SemaCXX/using-decl-templates.cpp +++ cfe/trunk/test/SemaCXX/using-decl-templates.cpp @@ -90,7 +90,7 @@ template struct A { }; template using APtr = A; // expected-note{{previous use is here}} - template struct APtr; // expected-error{{elaborated type refers to a type alias template}} + template struct APtr; // expected-error{{type alias template 'APtr' cannot be referenced with a struct specifier}} } namespace DontDiagnoseInvalidTest { Index: cfe/trunk/test/SemaTemplate/template-id-expr.cpp =================================================================== --- cfe/trunk/test/SemaTemplate/template-id-expr.cpp +++ cfe/trunk/test/SemaTemplate/template-id-expr.cpp @@ -100,5 +100,5 @@ class C {}; template