[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