[PATCH] D124447: [clang-tidy] Add infrastructure support for running on project-level information

Whisperity via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 14 01:48:25 PDT 2022


whisperity added a comment.

In D124447#3645197 <https://reviews.llvm.org/D124447#3645197>, @sammccall wrote:

> In D124447#3608747 <https://reviews.llvm.org/D124447#3608747>, @aaron.ballman wrote:
>
>> In general, I think this is looking pretty close to good. Whether clang-tidy should get this functionality in this form or not is a bit less clear to me. *I* think it's helpful functionality and the current approach is reasonable, but I think it might be worthwhile to have a community RFC to see if others agree. CC @alexfh in case he wants to make a code owner decision instead.
>
> +1 to a need for the RFC here, there are ecosystem questions that I think should be addressed first but aren't in scope of this patch.

I will attempt to write a concise response to this today (and tomorrow)! Sorry, I was away travelling to and doing post-action bureaucracy of conferences the past few weeks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124447



More information about the cfe-commits mailing list