From: Fariborz Jahanian Date: Tue, 8 Nov 2011 22:51:27 +0000 (+0000) Subject: objc: Don't crash on missing @interface decl. X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=90f7b62aba1ae2d41bfe2286c210b027fc62681f;p=clang objc: Don't crash on missing @interface decl. // rdar://10415026 git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@144143 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Sema/SemaExpr.cpp b/lib/Sema/SemaExpr.cpp index c6a5f1ec6f..4ba294cb8a 100644 --- a/lib/Sema/SemaExpr.cpp +++ b/lib/Sema/SemaExpr.cpp @@ -1968,12 +1968,13 @@ Sema::LookupInObjCMethod(LookupResult &Lookup, Scope *S, } } else if (CurMethod->isInstanceMethod()) { // We should warn if a local variable hides an ivar. - ObjCInterfaceDecl *IFace = CurMethod->getClassInterface(); - ObjCInterfaceDecl *ClassDeclared; - if (ObjCIvarDecl *IV = IFace->lookupInstanceVariable(II, ClassDeclared)) { - if (IV->getAccessControl() != ObjCIvarDecl::Private || - IFace == ClassDeclared) - Diag(Loc, diag::warn_ivar_use_hidden) << IV->getDeclName(); + if (ObjCInterfaceDecl *IFace = CurMethod->getClassInterface()) { + ObjCInterfaceDecl *ClassDeclared; + if (ObjCIvarDecl *IV = IFace->lookupInstanceVariable(II, ClassDeclared)) { + if (IV->getAccessControl() != ObjCIvarDecl::Private || + IFace == ClassDeclared) + Diag(Loc, diag::warn_ivar_use_hidden) << IV->getDeclName(); + } } } diff --git a/test/SemaObjC/invalid-code.m b/test/SemaObjC/invalid-code.m index 9b7f2c8a97..6a1e28ae9b 100644 --- a/test/SemaObjC/invalid-code.m +++ b/test/SemaObjC/invalid-code.m @@ -19,3 +19,10 @@ void foo() { @throw (id)0 // expected-error{{expected ';' after @throw}} } +// +@class NSView; +@implementation IBFillView(IBFillViewIntegration) // expected-error {{cannot find interface declaration for 'IBFillView'}} +- (NSView *)ibDesignableContentView { + return self; +} +@end