[PATCH] [Clang/CodeGen] Prevent crash if destructor class is not accessible

[Reid Kleckner]

================
Comment at: lib/CodeGen/CGClass.cpp:1296-1302
@@ -1295,5 +1295,9 @@
 {
+  // Give up if the destructor is not accessible.
+  if (!BaseClassDecl->getDestructor())
+    return false;
+
   // If the destructor is trivial we don't have to check anything else.
   if (BaseClassDecl->hasTrivialDestructor())
     return true;
 
----------------
I suspect you want to reorder these checks. The triviality check is a bittest, and getDestructor() may perform lookup.

http://reviews.llvm.org/D10508

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/