[clang] [clang-tools-extra] [clang-tidy][use-internal-linkage]fix false positives for ExportDecl (PR #117901)

Julian Schmidt via cfe-commits cfe-commits at lists.llvm.org
Sat Nov 30 12:51:27 PST 2024


5chmidti wrote:

> > Please add the matcher to https://github.com/llvm/llvm-project/blob/main/clang/lib/ASTMatchers/Dynamic/Registry.cpp as well (clang-query).
> 
> i will do it separately to avoid this pr become bigger.

I think you only have to add it into that file for it to work, and IMO a single PR for this change would be better. But as long as it gets added, I'm fine with it

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


More information about the cfe-commits mailing list