[PATCH] D88831: [clang-tidy] Remove obsolete checker google-runtime-references
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Oct 5 12:53:51 PDT 2020
aaron.ballman accepted this revision.
aaron.ballman added a comment.
LGTM to remove rather than move into `readability` unless someone has a good argument as to why it belongs there. I don't think this pattern makes code more readable in general as it recommends using a more verbose syntax (pointers, which require dereferencing) over a safer, concise syntax (references, which carry semantic information about nullness).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D88831/new/
https://reviews.llvm.org/D88831
More information about the cfe-commits
mailing list