jvoung wrote:
> AST_MATCHER_P_OVERLOAD
Ah, missed the part where the macro is generating the `internal` namespaces. I
don't think we want to move to the top of the file exactly (the instantiation
refers to some functions defined right above, so would need fwd decls?), but we
can change the namespacing inline `} ... {`.
There seem to be other recommendations in the header
https://github.com/llvm/llvm-project/blob/6854ad90e39e9d119c990043f573db7157a2b097/clang/include/clang/ASTMatchers/ASTMatchersMacros.h#L26
like putting in `clang::ast_matchers::unnamed...` (though I'm not sure if they
mean exactly `clang::ast_matchers`.
Let me try and check with the team.
https://github.com/llvm/llvm-project/pull/113601
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
