commit 2e34bbd8e842c6c0fb5adaaafcbf90d971d3c114
author Richard Smith <richard-llvm@metafoo.co.uk> Wed Aug 08 00:42:42 2018 +0000
committer Richard Smith <richard-llvm@metafoo.co.uk> Wed Aug 08 00:42:42 2018 +0000
tree 2194c4084a36bd9e0e7a042ec66ce133fbf6226e
parent ed2f9a6094ef2679fc8d9d70001c4f5b111c738d

PR38286: Don't crash when attempting to define a constructor for an
incomplete class template.

llvm-svn: 339210

clang/lib/Sema/SemaExprCXX.cpp

diff --git a/clang/lib/Sema/SemaExprCXX.cpp b/clang/lib/Sema/SemaExprCXX.cpp
index 22f1328..c851a81 100644
--- a/clang/lib/Sema/SemaExprCXX.cpp
+++ b/clang/lib/Sema/SemaExprCXX.cpp
@@ -113,9 +113,15 @@
       break;
     }
   }
-  if (!InjectedClassName && CurClass->isInvalidDecl())
+  if (!InjectedClassName) {
+    if (!CurClass->isInvalidDecl()) {
+      // FIXME: RequireCompleteDeclContext doesn't check dependent contexts
+      // properly. Work around it here for now.
+      Diag(SS.getLastQualifierNameLoc(),
+           diag::err_incomplete_nested_name_spec) << CurClass << SS.getRange();
+    }
     return ParsedType();
-  assert(InjectedClassName && "couldn't find injected class name");
+  }
 
   QualType T = Context.getTypeDeclType(InjectedClassName);
   DiagnoseUseOfDecl(InjectedClassName, NameLoc);