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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
