From d1ad5e5d6c895f809ada5b420060b2ec0b48567b Mon Sep 17 00:00:00 2001 From: Anna Zaks Date: Tue, 6 Nov 2012 04:20:54 +0000 Subject: [PATCH] [analyzer] Remove isWithinInlined. It's been replaced with inTopFrame(). Thanks Jordan. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@167438 91177308-0d34-0410-b5e6-96231b3b80d8 --- .../clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h | 5 ----- lib/StaticAnalyzer/Checkers/BasicObjCFoundationChecks.cpp | 2 +- 2 files changed, 1 insertion(+), 6 deletions(-) diff --git a/include/clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h b/include/clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h index b0f1c5897d..da7e6de735 100644 --- a/include/clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h +++ b/include/clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h @@ -130,11 +130,6 @@ public: /// Return true if the current LocationContext has no caller context. bool inTopFrame() const { return getLocationContext()->inTopFrame(); } - /// Returns true if the predecessor is within an inlined function/method. - bool isWithinInlined() { - return (getStackFrame()->getParent() != 0); - } - BugReporter &getBugReporter() { return Eng.getBugReporter(); } diff --git a/lib/StaticAnalyzer/Checkers/BasicObjCFoundationChecks.cpp b/lib/StaticAnalyzer/Checkers/BasicObjCFoundationChecks.cpp index e170aa56c5..0dd64780f1 100644 --- a/lib/StaticAnalyzer/Checkers/BasicObjCFoundationChecks.cpp +++ b/lib/StaticAnalyzer/Checkers/BasicObjCFoundationChecks.cpp @@ -763,7 +763,7 @@ void ObjCNonNilReturnValueChecker::checkPostObjCMessage(const ObjCMethodCall &M, // since 'nil' is rarely returned in practice, we should not warn when the // caller to the defensive constructor uses the object in contexts where // 'nil' is not accepted. - if (C.isWithinInlined() && M.getDecl() && + if (!C.inTopFrame() && M.getDecl() && M.getDecl()->getMethodFamily() == OMF_init && M.isReceiverSelfOrSuper()) { State = assumeExprIsNonNull(M.getOriginExpr(), State, C); -- 2.40.0