[clang] [Clang] [NFC] Introduce `DynamicRecursiveASTVisitor` (PR #110040)

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 5 05:24:05 PST 2024


https://github.com/AaronBallman approved this pull request.

I think these are a net positive change; @nikic are you comfortable with the performance numbers? I feel like the changes come awfully close to falling out in the noise aside from the positive improvements with building Clang, but you look at compile time performance numbers more than I do, so you may have a better sense for whether the changes are acceptable.

Signing off with a LGTM, but please wait to land until @nikic has weighed in.

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


More information about the cfe-commits mailing list