[cfe-commits] r160768 - in /cfe/trunk: include/clang/StaticAnalyzer/Core/PathSensitive/Calls.h include/clang/StaticAnalyzer/Core/PathSensitive/ExprEngine.h lib/StaticAnalyzer/Core/Calls.cpp lib/StaticAnalyzer/Core/ExprEngineCallAndReturn.cpp lib/StaticAnalyzer/Core/ExprEngineObjC.cpp test/Analysis/inlining/ test/Analysis/inlining/InlineObjCClassMethod.m

[Jordan Rose]

On Jul 26, 2012, at 9:29 , jahanian <fjahanian at apple.com> wrote:

>> -    for (Decl::redecl_iterator I = MD->redecls_begin(), E = MD->redecls_end();
>> -         I != E; ++I) {
>> -      if (cast<ObjCMethodDecl>(*I)->isThisDeclarationADefinition())
>> -        return *I;
>> +    const ObjCMessageExpr *E = getOriginExpr();
> 
> getOriginExpr may return null. You should check for that.

Right now it's safe to assume all ObjCMethodCalls have origin expressions, because all our implicit message sends are already abstracted using PseudoObjectExpr. I guess that may change in the future.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120726/d593d822/attachment.html>