<p dir="ltr">Test case(s)?</p>
<div class="gmail_quote">On Sep 2, 2013 2:49 AM, "Pavel Labath" <<a href="mailto:labath@google.com">labath@google.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi jordan_rose,<br>
<br>
rsmith suggested using CXXRecordDecl::isEmpty() instead, as it "also checks for<br>
virtual functions/virtual bases, and handles the case where the only fields are<br>
zero-length bitfields"<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D1573" target="_blank">http://llvm-reviews.chandlerc.com/D1573</a><br>
<br>
Files:<br>
  lib/StaticAnalyzer/Core/RegionStore.cpp<br>
<br>
Index: lib/StaticAnalyzer/Core/RegionStore.cpp<br>
===================================================================<br>
--- lib/StaticAnalyzer/Core/RegionStore.cpp<br>
+++ lib/StaticAnalyzer/Core/RegionStore.cpp<br>
@@ -1836,7 +1836,7 @@<br>
   if (!RD->field_empty())<br>
     return false;<br>
   if (const CXXRecordDecl *CRD = dyn_cast<CXXRecordDecl>(RD))<br>
-    return CRD->getNumBases() == 0;<br>
+    return CRD->isEmpty();<br>
   return true;<br>
 }<br>
<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>