https://github.com/zeyi2 requested changes to this pull request.
The majority of the fix looks good. Thanks for the fix! However, you still need to document the changes in [ReleaseNotes.rst, Changes in existing checks](https://github.com/llvm/llvm-project/blob/72f3995363eb7fe01df4eb54b3abb6cef3ee1568/clang-tools-extra/docs/ReleaseNotes.rst?plain=1#L351). Please remember to keep the alphabetical order :) https://github.com/llvm/llvm-project/pull/172220 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
