[PATCH] D121797: [clang][dataflow] Add modeling of Chromium's CHECK functionality
Yitzhak Mandelbaum via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 18 09:23:44 PDT 2022
ymandel added a comment.
In D121797#3392444 <https://reviews.llvm.org/D121797#3392444>, @xbolva00 wrote:
> Why this should be maintained and developed by LLVM/Clang developers and not by Chromium?
That's a good question. I think the short answer, that skirts around the issue, is that this is targeted for use in an upcoming clang-tidy check, and we don't have any framework for clients developing their own pluggable models for individual checks. Chromium could find some way to patch their clang-tidy source, I suppose, but we'd rather not encourage that. This framework is new and under development and the benefit of accomodating Chromium (in this very small way) seems worth the feedback from them applying it to their codebase.
That said, we are *also* in discussions with them to change their implement of CHECK, etc. so as to obviate the need for this model at all.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121797/new/
https://reviews.llvm.org/D121797
More information about the cfe-commits
mailing list