From 5808eb802941f4cb06bb4667d600b88a9251b8ff Mon Sep 17 00:00:00 2001 From: Anna Zaks Date: Mon, 30 Jul 2012 23:48:36 +0000 Subject: [PATCH] [analyzer] Handle inlining of instance calls to super. Use self-init.m for testing. (It used to have a bunch of failing tests with dynamic inlining turned on.) llvm-svn: 161012 --- clang/lib/StaticAnalyzer/Core/CallEvent.cpp | 26 +++++++++++++-------- clang/test/Analysis/self-init.m | 1 + 2 files changed, 17 insertions(+), 10 deletions(-) diff --git a/clang/lib/StaticAnalyzer/Core/CallEvent.cpp b/clang/lib/StaticAnalyzer/Core/CallEvent.cpp index ad5a104aa05e..bfc00ed2e0ea 100644 --- a/clang/lib/StaticAnalyzer/Core/CallEvent.cpp +++ b/clang/lib/StaticAnalyzer/Core/CallEvent.cpp @@ -592,21 +592,27 @@ ObjCMessageKind ObjCMethodCall::getMessageKind() const { const Decl *ObjCMethodCall::getRuntimeDefinition() const { const ObjCMessageExpr *E = getOriginExpr(); - Selector Sel = E->getSelector(); assert(E); + Selector Sel = E->getSelector(); if (E->isInstanceMessage()) { - const MemRegion *Receiver = getReceiverSVal().getAsRegion(); - DynamicTypeInfo TI = getState()->getDynamicTypeInfo(Receiver); - const ObjCObjectPointerType *T = - dyn_cast(TI.getType().getTypePtr()); - if (!T) - return 0; - if (ObjCInterfaceDecl *IDecl = T->getInterfaceDecl()) { - // Find the method implementation. - return IDecl->lookupPrivateMethod(Sel); + + // Find the the receiver type. + const ObjCObjectPointerType *ReceiverT = 0; + QualType SupersType = E->getSuperType(); + if (!SupersType.isNull()) { + ReceiverT = cast(SupersType.getTypePtr()); + } else { + const MemRegion *Receiver = getReceiverSVal().getAsRegion(); + DynamicTypeInfo TI = getState()->getDynamicTypeInfo(Receiver); + ReceiverT = dyn_cast(TI.getType().getTypePtr()); } + // Lookup the method implementation. + if (ReceiverT) + if (ObjCInterfaceDecl *IDecl = ReceiverT->getInterfaceDecl()) + return IDecl->lookupPrivateMethod(Sel); + } else { // This is a class method. // If we have type info for the receiver class, we are calling via diff --git a/clang/test/Analysis/self-init.m b/clang/test/Analysis/self-init.m index 10b0c4da47b6..b0c51a2b37e1 100644 --- a/clang/test/Analysis/self-init.m +++ b/clang/test/Analysis/self-init.m @@ -1,3 +1,4 @@ +// RUN: %clang_cc1 -analyze -analyzer-checker=osx.cocoa.SelfInit -fobjc-default-synthesize-properties -analyzer-ipa=dynamic -fno-builtin %s -verify // RUN: %clang_cc1 -analyze -analyzer-checker=osx.cocoa.SelfInit -fobjc-default-synthesize-properties -fno-builtin %s -verify @class NSZone, NSCoder; -- GitLab