From 4bbae38abf4145b81a3bf8658968fdaa2a8941c1 Mon Sep 17 00:00:00 2001 From: Reid Kleckner Date: Thu, 28 Mar 2013 22:15:11 +0000 Subject: [PATCH] [sema] Check the result of getAsCXXRecordDecl() to fix the build 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 | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/Sema/SemaType.cpp b/lib/Sema/SemaType.cpp index cbd8c31fa7..06004bd00c 100644 --- a/lib/Sema/SemaType.cpp +++ b/lib/Sema/SemaType.cpp @@ -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()) { + if (!RD || !RD->hasAttr()) { // 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. -- 2.40.0