[PATCH] D146922: [clang-tidy] Fix false positve for defaulted move constructor in performance-noexcept-move-constructor

André Schackier via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 30 09:18:52 PDT 2023


AMS21 added inline comments.


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:260
 
+- Fixed an issue in :doc:`performance-noexcept-move-constructor
+  <clang-tidy/checks/performance/noexcept-move-constructor>` which resulted in false
----------------
Eugene.Zelenko wrote:
> Please keep alphabetical order (by check name) in this section.
Sure but I'm a bit confused. I assuming the check name is without the prefix so in this case `noexcept-move-constructor`. So it should be after `readability-avoid-underscore-in-googletest-name`. But why then is `use-after-move` sorted before `identifier-naming`. 
I'm probably missing something very obvious.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146922



More information about the cfe-commits mailing list