Rip out TemplateIdRefExpr and make UnresolvedLookupExpr and

DependentScopeDeclRefExpr support storing templateids.  Unite the common   
code paths between ActOnDeclarationNameExpr and ActOnTemplateIdExpr.

This gets us to a point where we don't need to store function templates in
the AST using TemplateNames, which is critical to ripping out OverloadedFunction.

Also resolves a few FIXMEs.

llvm-svn: 89785

1 files changed, 1 insertions, 5 deletions

diff --git a/clang/lib/Sema/SemaLookup.cpp b/clang/lib/Sema/SemaLookup.cpp
index 08e9dc3c631..8f0982740d4 100644
--- a/clang/lib/Sema/SemaLookup.cpp
+++ b/clang/lib/Sema/SemaLookup.cpp
@@ -1544,11 +1544,7 @@ Sema::FindAssociatedClassesAndNamespaces(Expr **Args, unsigned NumArgs,
 
     if (UnresolvedLookupExpr *ULE = dyn_cast<UnresolvedLookupExpr>(Arg))
       Functions.append(ULE->decls_begin(), ULE->decls_end());
-    else if (TemplateIdRefExpr *TIRE = dyn_cast<TemplateIdRefExpr>(Arg)) {
-      TemplateName TName = TIRE->getTemplateName();
-      OverloadedFunctionDecl *Ovl = TName.getAsOverloadedFunctionDecl();
-      Functions.append(Ovl->function_begin(), Ovl->function_end());
-    } else
+    else
       continue;
 
     for (llvm::SmallVectorImpl<NamedDecl*>::iterator I = Functions.begin(),