[PATCH] D96281: [clang-tidy] Add options to flag individual core increments and to ignore macros to readability-function-cognitive-complexity check.
Roman Lebedev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Mar 2 01:20:53 PST 2021
lebedev.ri added a comment.
Once again, can we please split this into two patches?
I'm quite happy with the `DescribeBasicIncrements` stuff, that can be merged right now.
But i'm not quite sold on the macro stuff. We don't have that for readability-function-size,
nor does the CC spec say anything about macros, nor do other implementations allow to discount them.
To take it to extreme, one potentially could wrap the entire function into a macro and instantiate it once,
and end up with a zero complexity, even though it's obviously not so..
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96281/new/
https://reviews.llvm.org/D96281
More information about the cfe-commits
mailing list