[PATCH] D72217: [clang-tidy] Added readability-qualified-auto check

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 8 02:11:32 PST 2020


njames93 added a comment.

In D72217#1809212 <https://reviews.llvm.org/D72217#1809212>, @Eugene.Zelenko wrote:

> By the word, will be interesting to see results of this check run on LLVM code. Probably results should be split on modules.


So ran it on clang and clang-tidy just crashed, gonna debug it see whats happening



================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:146
 
+- New alias :doc:`llvm-qualified-auto
+  <clang-tidy/checks/llvm-qualified-auto>` to
----------------
Eugene.Zelenko wrote:
> Please move alias entry into aliases section. See previous release for proper order.
I looked at the previous release notes and from what i can see, all alias checks are next to each other in alphabetical order, but there is no special placement of the alias checks in the release notes, they just start at an arbitrary position. 


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72217/new/

https://reviews.llvm.org/D72217





More information about the cfe-commits mailing list