[clang] [clang] Adds a RecursiveASTEnterExitVisitor data type (PR #136136)

via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 17 08:23:17 PDT 2025


https://github.com/Sirraide requested changes to this pull request.

I’m sorry, but without a *very* strong motivation and concrete use case for this, I’m going to NAK this addition: as a matter of fact, I’ve actively been working on *removing* uses of `RecursiveASTVisitor` in favour of `DynamicRecursiveASTVisitor` for a while now because of the former’s rather abysmal impact on both compile-times and binary size—the last thing we want, unless it it *absolutely necessary* for some reason, is add *another* CRTP-like recursive AST visitor.

https://github.com/llvm/llvm-project/pull/136136


More information about the cfe-commits mailing list