[PATCH] D93182: [clang-tidy] Add linux kernel log functions checker

Bill Wendling via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 16 13:25:57 PDT 2022


void added a comment.

In D93182#2451509 <https://reviews.llvm.org/D93182#2451509>, @lebedev.ri wrote:

> Out of curiocity, are all checkpatch diagnostics (and something else)?
> are planned to be ported into being clang-tidy checks?
> How many will checks will that be?
> I'll be slightly worried if `linuxkernel` module
> ends up having more checks than all other modules combined..

In my opinion, we shouldn't be concerned about the number of checks, especially with a code base as large and complex as Linux. Linux also has security issues that other pieces of software don't have, which need detailed checks.

If the number of checks is considered too much, what other options should we go with?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93182



More information about the cfe-commits mailing list