https://github.com/vbvictor commented:
I feel this shouldn't be in `ClangTidyCheck` class. The class should store minimal info for analysis and shouldn't care if the check is deprecated or not. I believe we currently break [SRP](https://en.wikipedia.org/wiki/Single-responsibility_principle). The better place would be in utils with something like: ```cpp isDeprecatedAlias(Context, DeprecatedName, TargetName) ``` https://github.com/llvm/llvm-project/pull/189522 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
