[clang-tools-extra] [clang-tidy][readability] Ignore macros in function-size check (PR #199549)
via cfe-commits
cfe-commits at lists.llvm.org
Tue Jun 2 00:32:21 PDT 2026
savchart wrote:
Added a test for this case with one extra non-macro statement.
With only `BODY`, the function is skipped: after ignoring the macro-expanded statement, `FI.Statements == 0`, and `FunctionSizeCheck::check()` returns early.
This still tests the intended behavior: `IgnoreMacros` does not affect `ParameterThreshold`, since parameter counting is done in `check()`, not during AST traversal.
https://github.com/llvm/llvm-project/pull/199549
More information about the cfe-commits
mailing list