[PATCH] D36836: [clang-tidy] Implement readability-function-cognitive-complexity check

[Roman Lebedev via Phabricator via cfe-commits]

lebedev.ri added a comment.

In https://reviews.llvm.org/D36836#845417, @JonasToth wrote:

> Did I run it over clang or athletic like that? I would be interested in an overall output.


No, not yet.
But if it is to be enabled for LLVM repos by default, then the `Threshold` will most definitively need adjusting in a `.clang-tidy` :)

> I added my thoughts.

Should be fixed.

> This check seems very valuable.

Thank you :)
I hope others agree...


Repository:
  rL LLVM

https://reviews.llvm.org/D36836