[llvm-branch-commits] [clang] [clang][test] add testing for the AST matcher reference (PR #94248)
Aaron Ballman via llvm-branch-commits
llvm-branch-commits at lists.llvm.org
Tue Jul 2 11:47:57 PDT 2024
AaronBallman wrote:
> Thanks.
>
> > ... how this impacts build times for Clang itself? I'm assuming that if ASTMatchers.h isn't modified, CMake won't re-run generate_ast_matcher_doc_tests.py and so the compile time performance hit is only on full rebuilds or when changing the header?
>
> The 'state' of the generated file is only checked when the `ASTMatchersTests` target is built, because it's the only thing that depends on the generated file. And the file is only generated when: the file does not exist or `ASTMatchers.h` has changed (excluding transitive changes in includes) or `generate_ast_matcher_doc_tests.py` has changed.
Excellent, thank you for the confirmation! That sounds reasonable to me.
https://github.com/llvm/llvm-project/pull/94248
More information about the llvm-branch-commits
mailing list