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

Make the selection of type declarations in Sema::getTypeName 

deterministic when faced with an ambiguity. This eliminates the
annoying test/SemaCXX/using-directive.cpp failure.

llvm-svn: 68952
This commit is contained in:
Douglas Gregor 2009-04-13 15:14:38 +00:00
parent 092b8b6fdb
commit 712a351c42
2 changed files with 6 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -85,8 +85,10 @@ Sema::TypeTy *Sema::getTypeName(IdentifierInfo &II, SourceLocation NameLoc,
for (LookupResult::iterator Res = Result.begin(), ResEnd = Result.end();
Res != ResEnd; ++Res) {
if (isa<TypeDecl>(*Res) || isa<ObjCInterfaceDecl>(*Res)) {
IIDecl = *Res;
break;
if (!IIDecl ||
(*Res)->getLocation().getRawEncoding() <
IIDecl->getLocation().getRawEncoding())
IIDecl = *Res;
}
}

7
clang/test/SemaCXX/using-directive.cpp
View File

@ -1,5 +1,4 @@
// RUN: clang-cc -fsyntax-only -verify %s
// XFAIL
namespace A {
short i; // expected-note 2{{candidate found by name lookup is 'A::i'}}
@ -95,8 +94,7 @@ namespace OneFunction {
}
namespace TwoTag {
struct X; // expected-note{{candidate found by name lookup is 'TwoTag::X'}} \
// expected-note{{forward declaration}}
struct X; // expected-note{{candidate found by name lookup is 'TwoTag::X'}}
}
namespace FuncHidesTagAmbiguity {
@ -105,7 +103,6 @@ namespace FuncHidesTagAmbiguity {
using namespace TwoTag;
void test() {
(void)X(); // expected-error{{reference to 'X' is ambiguous}} \
// FIXME: expected-error{{invalid use of incomplete type}}
(void)X(); // expected-error{{reference to 'X' is ambiguous}}
}
}