[PATCH] D138031: Add clang-tidy check for missing move constructors

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 15 09:56:09 PST 2022


njames93 added a comment.

What is the purpose of this check when we have cppcoreguidelines-special-member-functions <https://clang.llvm.org/extra/clang-tidy/checks/cppcoreguidelines/special-member-functions.html>. 
Granted that check won't emit replacements, however the replacements that this check generates would often not improve performance or potentially introduce a bug.

I know its not nearly the same, but clangd can also fix this interactively with its special members <https://github.com/llvm/llvm-project/blob/main/clang-tools-extra/clangd/refactor/tweaks/SpecialMembers.cpp> refactor


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138031



More information about the cfe-commits mailing list