[clang] [clang-tools-extra] [analyzer] Remove alpha.core.IdenticalExpr Checker (PR #114715)

DonĂ¡t Nagy via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 4 05:22:58 PST 2024


https://github.com/NagyDonat commented:

Thanks for implementing this cleanup commit!

I like that you ported the tests from the old checker to the tidy checks. However, as these test files are very large, perhaps it would be better to put these moved tests into stand-alone files instead of adding them at the end of existing files. (@ tidy developers: what would you prefer?)

Do I understand it correctly that the two tidy checkers report all the issues that were reported by alpha.core.IdenticalExpr? (Unfortunately github cannot show a clear diff between the old and new version of the moved code.)

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


More information about the cfe-commits mailing list