<div style="font-family: arial, helvetica, sans-serif"><font size="2"><div class="gmail_quote">On Thu, Jun 14, 2012 at 8:51 AM, Daniel Jasper <span dir="ltr"><<a href="mailto:djasper@google.com" target="_blank">djasper@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="font-family:arial,helvetica,sans-serif"><font size="2"><div>Look at incomplete FunctionTemplateDecls in order to determine whether a CXXRecordDecl is complete. Fixes Bug 13086.</div>
</font></div></blockquote><div><br></div><div>LGTM</div><div><br></div><div>For future reference, please attach the patch rather than pasting it -- email clients like to line wrap pasted patches... </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="font-family:arial,helvetica,sans-serif"><font size="2"><div><br></div><div>diff --git a/lib/Sema/Sema.cpp b/lib/Sema/Sema.cpp</div>
<div><div>index e0d405d..39338f9 100644</div><div>--- a/lib/Sema/Sema.cpp</div><div>+++ b/lib/Sema/Sema.cpp</div><div>@@ -444,6 +444,8 @@ static bool MethodsAndNestedClassesComplete(const CXXRecordDecl *RD,</div><div>        I != E && Complete; ++I) {</div>

<div>     if (const CXXMethodDecl *M = dyn_cast<CXXMethodDecl>(*I))</div><div>       Complete = M->isDefined() || (M->isPure() && !isa<CXXDestructorDecl>(M));</div><div>+    else if (const FunctionTemplateDecl *F = dyn_cast<FunctionTemplateDecl>(*I))</div>

<div>+      Complete = F->getTemplatedDecl()->isDefined();</div><div>     else if (const CXXRecordDecl *R = dyn_cast<CXXRecordDecl>(*I)) {</div><div>       if (R->isInjectedClassName())</div><div>         continue;</div>

<div>diff --git a/test/SemaCXX/warn-unused-member.cpp b/test/SemaCXX/warn-unused-member.cpp</div><div>index 00de8a9..9ec9df6 100644</div><div>--- a/test/SemaCXX/warn-unused-member.cpp</div><div>+++ b/test/SemaCXX/warn-unused-member.cpp</div>

<div>@@ -142,6 +142,13 @@ class EverythingUsed {</div><div>   int by_initializer_;</div><div> };</div><div> </div><div>+namespace templates {</div><div>+class B {</div><div>+  template <typename T> void f(T t);</div>

<div>+  int a;</div><div>+};</div><div>+}  // namespace templates</div><div>+</div><div> namespace mutual_friends {</div><div> // Undefined methods make mutual friends undefined.</div><div> class A {</div></div><div><br>
</div>
</font></div>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></font></div>