[clang] [mutation analyzer] support mutation analysis for pointee (PR #118593)
Congcong Cai via cfe-commits
cfe-commits at lists.llvm.org
Tue Jan 21 05:46:18 PST 2025
================
@@ -112,6 +115,57 @@ AST_MATCHER_P(Stmt, canResolveToExpr, const Stmt *, Inner) {
return canExprResolveTo(Exp, Target);
}
+// use class member to store data can reduce stack usage to avoid stack overflow
+// when recursive call.
+class ExprPointeeResolve {
+ const Expr *T;
+
+ bool resolveExpr(const Expr *E) {
+ if (E == nullptr)
+ return false;
+ if (E == T)
+ return true;
+
+ if (const auto *BO = dyn_cast<BinaryOperator>(E)) {
+ if (BO->isAdditiveOp())
+ return (resolveExpr(BO->getLHS()) || resolveExpr(BO->getRHS()));
+ if (BO->isCommaOp())
+ return resolveExpr(BO->getRHS());
+ return false;
+ }
+
+ if (const auto *PE = dyn_cast<ParenExpr>(E))
+ return resolveExpr(PE->getSubExpr());
+
+ if (const auto *ICE = dyn_cast<ImplicitCastExpr>(E)) {
+ const CastKind kind = ICE->getCastKind();
+ if (kind == CK_LValueToRValue || kind == CK_DerivedToBase ||
+ kind == CK_UncheckedDerivedToBase)
+ return resolveExpr(ICE->getSubExpr());
+ return false;
+ }
+
----------------
HerrCai0907 wrote:
`ExplicitCastExpr` should be treat as different case instead of directly resolvable.
Because explicit cast only need to fulfill requirement of case itself.
e.g.
In `dynamicCast<Derived*>(BasePtr)`, we don't need to analyze whether the whole expr is mutable. Here we only need to check the constness compatibility between `decltype(BasePtr)` and `Derived*`. Which is already checked in cast part of `findPointeeToNonConst`
https://github.com/llvm/llvm-project/pull/118593
More information about the cfe-commits
mailing list