]> granicus.if.org Git - clang/commitdiff
[sema] Check the result of getAsCXXRecordDecl() to fix the build
authorReid Kleckner <reid@kleckner.net>
Thu, 28 Mar 2013 22:15:11 +0000 (22:15 +0000)
committerReid Kleckner <reid@kleckner.net>
Thu, 28 Mar 2013 22:15:11 +0000 (22:15 +0000)
I'm not 100% sure what should happen here to find the real
CXXRecordDecl.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@178297 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Sema/SemaType.cpp

index cbd8c31fa7e3b17a4aca86c8451fb79e58b4600a..06004bd00c5bf8c423c53735e77115f340489258 100644 (file)
@@ -1733,12 +1733,12 @@ QualType Sema::BuildMemberPointerType(QualType T, QualType Class,
   // when passing pointers between TUs that disagree about the size.
   if (Context.getTargetInfo().getCXXABI().isMicrosoft()) {
     CXXRecordDecl *RD = Class->getAsCXXRecordDecl();
-    if (!RD->hasAttr<MSInheritanceAttr>()) {
+    if (!RD || !RD->hasAttr<MSInheritanceAttr>()) {
       // Lock in the inheritance model on the first use of a member pointer.
       // Otherwise we may disagree about the size at different points in the TU.
       // FIXME: MSVC picks a model on the first use that needs to know the size,
       // rather than on the first mention of the type, e.g. typedefs.
-      if (RequireCompleteType(Loc, Class, 0) && !RD->isBeingDefined()) {
+      if (RequireCompleteType(Loc, Class, 0) && RD && !RD->isBeingDefined()) {
         // We know it doesn't have an attribute and it's incomplete, so use the
         // unspecified inheritance model.  If we're in the record body, we can
         // figure out the inheritance model.