================ @@ -222,6 +222,49 @@ bool Expr::isFlexibleArrayMemberLike( IgnoreTemplateOrMacroSubstitution); } +namespace { + +/// MemberExprVisitor - Find the MemberExpr through all of the casts, array +/// subscripts, and unary ops. This intentionally avoids all of them because +/// we're interested only in the MemberExpr to check if it's a flexible array +/// member. +class MemberExprVisitor + : public ConstStmtVisitor<MemberExprVisitor, const Expr *> { +public: + //===--------------------------------------------------------------------===// + // Visitor Methods + //===--------------------------------------------------------------------===// + + const Expr *Visit(const Expr *E) { + return ConstStmtVisitor<MemberExprVisitor, const Expr *>::Visit(E); + } + const Expr *VisitStmt(const Stmt *S) { return nullptr; } ---------------- Sirraide wrote:
I guess this is for deranged cases like `__builtin_get_counted_by(({ foo.x; }))`? It seems reasonable not to support that, but we should probably have tests for it. https://github.com/llvm/llvm-project/pull/102549 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits