[PATCH] D7982: [clang-tidy] Add readability-duplicate-include check

Richard via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 20 11:58:01 PST 2022


LegalizeAdulthood added a comment.

> ! In D7982#3258888 <https://reviews.llvm.org/D7982#3258888>, @carlosgalvezp wrote:
>  I need to familiarize myself better with the `Loc` manipulation
>  code to be able to review the free-standing function but otherwise
>  looks reasonable.

The "clang internals" documentation does a good job of explaining the
location classes:
https://clang.llvm.org/docs/InternalsManual.html#the-sourcelocation-and-sourcemanager-classes

This reminds me that I've been thinking we need to write some
documentation for clang-tidy check authors to help orient them
on the various parts involved in writing a check.


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

https://reviews.llvm.org/D7982



More information about the cfe-commits mailing list