summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDaniel Jasper <djasper@google.com>2012-06-14 20:56:06 +0000
committerDaniel Jasper <djasper@google.com>2012-06-14 20:56:06 +0000
commite99c2bf8aa61d1e5e813e3eeedfc15608812003b (patch)
tree8e006b5aadc768c36a3cf9130e4b30787bdc2e3b
parent1b420ac4c800136790d7f5c0cacc022f769d93f5 (diff)
downloadbcm5719-llvm-e99c2bf8aa61d1e5e813e3eeedfc15608812003b.tar.gz
bcm5719-llvm-e99c2bf8aa61d1e5e813e3eeedfc15608812003b.zip
Look at incomplete FunctionTemplateDecls in order to determine whether
a CXXRecordDecl is complete. Fixes Bug 13086. llvm-svn: 158469
-rw-r--r--clang/lib/Sema/Sema.cpp2
-rw-r--r--clang/test/SemaCXX/warn-unused-member.cpp7
2 files changed, 9 insertions, 0 deletions
diff --git a/clang/lib/Sema/Sema.cpp b/clang/lib/Sema/Sema.cpp
index e0d405d175f..39338f983e5 100644
--- a/clang/lib/Sema/Sema.cpp
+++ b/clang/lib/Sema/Sema.cpp
@@ -444,6 +444,8 @@ static bool MethodsAndNestedClassesComplete(const CXXRecordDecl *RD,
I != E && Complete; ++I) {
if (const CXXMethodDecl *M = dyn_cast<CXXMethodDecl>(*I))
Complete = M->isDefined() || (M->isPure() && !isa<CXXDestructorDecl>(M));
+ else if (const FunctionTemplateDecl *F = dyn_cast<FunctionTemplateDecl>(*I))
+ Complete = F->getTemplatedDecl()->isDefined();
else if (const CXXRecordDecl *R = dyn_cast<CXXRecordDecl>(*I)) {
if (R->isInjectedClassName())
continue;
diff --git a/clang/test/SemaCXX/warn-unused-member.cpp b/clang/test/SemaCXX/warn-unused-member.cpp
index 00de8a97aca..9ec9df66ec7 100644
--- a/clang/test/SemaCXX/warn-unused-member.cpp
+++ b/clang/test/SemaCXX/warn-unused-member.cpp
@@ -142,6 +142,13 @@ class EverythingUsed {
int by_initializer_;
};
+namespace templates {
+class B {
+ template <typename T> void f(T t);
+ int a;
+};
+} // namespace templates
+
namespace mutual_friends {
// Undefined methods make mutual friends undefined.
class A {
OpenPOWER on IntegriCloud