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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to