[PATCH] D137205: [clang-tidy] Add performance-unnecessary-copy-on-last-use check
Carlos Galvez via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 28 02:40:54 PST 2022
carlosgalvezp added a comment.
> by design clang-format only performs white-space changes
clang-format does support east/west const enforcement with the `QualifierAlignment` option. From experience, I strongly encourage repo owners to enable it repo-wide to avoid these kinds of discussions. Until then, I believe developers should keep consistency with the existing style in the codebase, regardless of personal views. This goes in line with the LLVM Coding Standards:
"If you are extending, enhancing, or bug fixing already implemented code, use the style that is already being used so that the source is uniform and easy to follow."
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D137205/new/
https://reviews.llvm.org/D137205
More information about the cfe-commits
mailing list