[PATCH] Instantiate incomplete class used in template method.

[Serge Pavlov]

Updated patch

Thank you for advice and explanation! Changed the patch accordingly.


http://reviews.llvm.org/D8281

Files:
  lib/Sema/SemaTemplateInstantiate.cpp
  lib/Sema/SemaTemplateInstantiateDecl.cpp
  test/SemaTemplate/instantiate-local-class.cpp

Index: lib/Sema/SemaTemplateInstantiate.cpp
===================================================================
--- lib/Sema/SemaTemplateInstantiate.cpp
+++ lib/Sema/SemaTemplateInstantiate.cpp
@@ -2796,6 +2796,12 @@
       isa<TemplateTemplateParmDecl>(D))
     return nullptr;
 
+  // If the declaration we are looking for is an incomplete type declaration, we
+  // may not have a chance to instantiate it.
+  if (const TagDecl *TD = dyn_cast<TagDecl>(D))
+    if (TD->getTypeForDecl()->isIncompleteType())
+      return nullptr;
+
   // If we didn't find the decl, then we either have a sema bug, or we have a
   // forward reference to a label declaration.  Return null to indicate that
   // we have an uninstantiated label.
Index: lib/Sema/SemaTemplateInstantiateDecl.cpp
===================================================================
--- lib/Sema/SemaTemplateInstantiateDecl.cpp
+++ lib/Sema/SemaTemplateInstantiateDecl.cpp
@@ -4399,6 +4399,15 @@
         isa<TemplateTemplateParmDecl>(D))
       return D;
 
+    // If the declaration we are looking for is an incomplete type declaration,
+    // we may not have a chance to instantiate it.
+    if (const TagDecl *TD = dyn_cast<TagDecl>(D))
+      if (!TD->isThisDeclarationADefinition() && !TD->isFreeStanding()) {
+        Decl *Inst = SubstDecl(D, CurContext, TemplateArgs);
+        CurrentInstantiationScope->InstantiatedLocal(D, Inst);
+        return cast<TypeDecl>(Inst);
+      }
+
     if (D->isInvalidDecl())
       return nullptr;
 
Index: test/SemaTemplate/instantiate-local-class.cpp
===================================================================
--- test/SemaTemplate/instantiate-local-class.cpp
+++ test/SemaTemplate/instantiate-local-class.cpp
@@ -194,3 +194,16 @@
   void f() { F<int>(); }
 };
 }
+
+namespace PR18653 {
+  template <class T> void f();
+  template <class T> struct str {
+    void m() {
+      f<class newclass>();
+    }
+  };
+  template struct str<int>;
+
+  template<typename T> void f() { void g(struct x); }
+  template void f<int>();
+}

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D8281.23254.patch
Type: text/x-patch
Size: 2048 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150405/fb185812/attachment.bin>