[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