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

[Eugene Zelenko via Phabricator via cfe-commits]

Eugene.Zelenko 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.  
----------------
Febbe wrote:
> 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?
> 
I meant that first sentence in documentation should be same as Release Notes record.

80-characters limit should be followed in code and documentation.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137205