================ @@ -163,10 +163,13 @@ bool isConstOwnerPtrMemberExpr(const clang::Expr *E) { if (OCE->getOperator() == OO_Star && OCE->getNumArgs() == 1) E = OCE->getArg(0); } - auto *ME = dyn_cast<MemberExpr>(E); - if (!ME) + const ValueDecl *D = nullptr; + if (auto *ME = dyn_cast<MemberExpr>(E)) + D = ME->getMemberDecl(); + else if (auto *IVR = dyn_cast<ObjCIvarRefExpr>(E)) + D = IVR->getDecl(); + else return false; - auto *D = ME->getMemberDecl(); if (!D) ---------------- t-rasmud wrote:
Is this check still needed? The `else` clause above should already return if D is a nullptr. https://github.com/llvm/llvm-project/pull/126353 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits