[PATCH] D137205: [clang-tidy] Add performance-unnecessary-copy-on-last-use check

[Fabian Keßler via Phabricator via cfe-commits]

Febbe marked 10 inline comments as done.
Febbe added inline comments.


================
Comment at: clang-tools-extra/docs/clang-tidy/checks/performance/unnecessary-copy-on-last-use.rst:6
+
+Finds variables of non-trivially-copyable types, that are used in a copy construction on their last usage and suggest to wrap the usage in a `std::move`.
+The usage just before an assignment is interpreted as last usage.  
----------------
Eugene.Zelenko wrote:
> Please synchronize with Release Notes, follow 80 characters limit and use double back-ticks for language constructs.
Do you mean, to add this short description to the release notes?
Is the 80 characters limit for the line, or the whole description?



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137205