summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEli Friedman <eli.friedman@gmail.com>2009-07-31 01:43:05 +0000
committerEli Friedman <eli.friedman@gmail.com>2009-07-31 01:43:05 +0000
commitdfbd0c4b0db17f50054dd7ce0d76cec48445780f (patch)
treeddfd2493e7def49ef62e2d7292a3f09cfa330bcb
parenta076d14514200507ba381bada8e23adf5e6ba17d (diff)
downloadbcm5719-llvm-dfbd0c4b0db17f50054dd7ce0d76cec48445780f.tar.gz
bcm5719-llvm-dfbd0c4b0db17f50054dd7ce0d76cec48445780f.zip
Make the check for the linkage of a template handle the case of nested
linkage specifications correctly. llvm-svn: 77653
-rw-r--r--clang/lib/Sema/SemaTemplate.cpp11
-rw-r--r--clang/test/SemaTemplate/nested-linkage.cpp3
2 files changed, 8 insertions, 6 deletions
diff --git a/clang/lib/Sema/SemaTemplate.cpp b/clang/lib/Sema/SemaTemplate.cpp
index 4264f32e9d4..29258575c1c 100644
--- a/clang/lib/Sema/SemaTemplate.cpp
+++ b/clang/lib/Sema/SemaTemplate.cpp
@@ -2123,13 +2123,12 @@ Sema::CheckTemplateDeclScope(Scope *S,
// A template-declaration can appear only as a namespace scope or
// class scope declaration.
DeclContext *Ctx = static_cast<DeclContext *>(S->getEntity());
- while (Ctx && isa<LinkageSpecDecl>(Ctx)) {
- if (cast<LinkageSpecDecl>(Ctx)->getLanguage() != LinkageSpecDecl::lang_cxx)
- return Diag(TemplateLoc, diag::err_template_linkage)
- << TemplateRange;
-
+ if (Ctx && isa<LinkageSpecDecl>(Ctx) &&
+ cast<LinkageSpecDecl>(Ctx)->getLanguage() != LinkageSpecDecl::lang_cxx)
+ return Diag(TemplateLoc, diag::err_template_linkage) << TemplateRange;
+
+ while (Ctx && isa<LinkageSpecDecl>(Ctx))
Ctx = Ctx->getParent();
- }
if (Ctx && (Ctx->isFileContext() || Ctx->isRecord()))
return false;
diff --git a/clang/test/SemaTemplate/nested-linkage.cpp b/clang/test/SemaTemplate/nested-linkage.cpp
new file mode 100644
index 00000000000..ffe9adc927d
--- /dev/null
+++ b/clang/test/SemaTemplate/nested-linkage.cpp
@@ -0,0 +1,3 @@
+// RUN: clang-cc -fsyntax-only -verify %s
+
+extern "C" { extern "C++" { template<class C> C x(); } }
OpenPOWER on IntegriCloud