[cfe-commits] r89089 - /cfe/trunk/lib/Sema/SemaLookup.cpp

John McCall rjmccall at apple.com
Tue Nov 17 02:36:42 PST 2009


Author: rjmccall
Date: Tue Nov 17 04:36:41 2009
New Revision: 89089

URL: http://llvm.org/viewvc/llvm-project?rev=89089&view=rev
Log:
Temporarily unbreak the clang-on-llvm tests. :)  Not going to fix unresolved
lookup all in a night.


Modified:
    cfe/trunk/lib/Sema/SemaLookup.cpp

Modified: cfe/trunk/lib/Sema/SemaLookup.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaLookup.cpp?rev=89089&r1=89088&r2=89089&view=diff

==============================================================================
--- cfe/trunk/lib/Sema/SemaLookup.cpp (original)
+++ cfe/trunk/lib/Sema/SemaLookup.cpp Tue Nov 17 04:36:41 2009
@@ -254,7 +254,6 @@
 
   bool Ambiguous = false;
   bool HasTag = false, HasFunction = false, HasNonFunction = false;
-  bool HasUnresolved = false;
 
   unsigned UniqueTagIndex = 0;
   
@@ -268,7 +267,7 @@
       // continue at this index).
       Decls[I] = Decls[--N];
     } else if (isa<UnresolvedUsingDecl>(D)) {
-      HasUnresolved = true;
+      // FIXME: support unresolved using decls
       Decls[I] = Decls[--N];
     } else {
       // Otherwise, do some decl type analysis and then continue.
@@ -288,13 +287,6 @@
     }
   }
 
-  // Postpone all other decisions if we have an unresolved decl, even
-  // if we can prove ambiguity.  We can probably do better than this.
-  if (HasUnresolved) {
-    ResultKind = LookupResult::FoundOverloaded;
-    return;
-  }
-
   // C++ [basic.scope.hiding]p2:
   //   A class name or enumeration name can be hidden by the name of
   //   an object, function, or enumerator declared in the same





More information about the cfe-commits mailing list