[cfe-commits] r139105 - /cfe/trunk/lib/Sema/SemaExpr.cpp

Francois Pichet pichet2000 at gmail.com
Sun Sep 4 16:00:48 PDT 2011


Author: fpichet
Date: Sun Sep  4 18:00:48 2011
New Revision: 139105

URL: http://llvm.org/viewvc/llvm-project?rev=139105&view=rev
Log:
Pass 0 instead of a empty TemplateArgumentListInfo when creating a CXXDependentScopeMemberExpr to handle a "this->" fixit (lookup into dependent bases of class template)

Otherwise the fixit doesn't really work for subsequent lookup.

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

Modified: cfe/trunk/lib/Sema/SemaExpr.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaExpr.cpp?rev=139105&r1=139104&r2=139105&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/SemaExpr.cpp (original)
+++ cfe/trunk/lib/Sema/SemaExpr.cpp Sun Sep  4 18:00:48 2011
@@ -1474,7 +1474,8 @@
                 CXXDependentScopeMemberExpr::Create(
                     Context, DepThis, DepThisType, true, SourceLocation(),
                     SS.getWithLocInContext(Context), NULL,
-                    R.getLookupNameInfo(), &TList);
+                    R.getLookupNameInfo(),
+                    ULE->hasExplicitTemplateArgs() ? &TList : 0);
             CallsUndergoingInstantiation.back()->setCallee(DepExpr);
           } else {
             // FIXME: we should be able to handle this case too. It is correct





More information about the cfe-commits mailing list