[clang] 767d34e - Disable misc-no-recursion checking in Clang

[Aaron Ballman via cfe-commits]

Author: Aaron Ballman
Date: 2021-05-27T10:39:04-04:00
New Revision: 767d34e3bdddef6c1871006dd0a2d06a4e1bcd5d

URL: https://github.com/llvm/llvm-project/commit/767d34e3bdddef6c1871006dd0a2d06a4e1bcd5d
DIFF: https://github.com/llvm/llvm-project/commit/767d34e3bdddef6c1871006dd0a2d06a4e1bcd5d.diff

LOG: Disable misc-no-recursion checking in Clang

We currently enable misc-no-recursion, but Clang uses recursion
intentionally in a fair number of places (like RecursiveASTVisitor).
Disabling this check reduces a noise in reviews that add new AST nodes,
like https://reviews.llvm.org/D103112#2780747 which has five CI
warnings that the author can do nothing about.

Added: 
    

Modified: 
    clang/.clang-tidy

Removed: 
    


################################################################################
diff  --git a/clang/.clang-tidy b/clang/.clang-tidy
index 5b425a712023..f517e9246cf8 100644
--- a/clang/.clang-tidy
+++ b/clang/.clang-tidy
@@ -1,4 +1,4 @@
-Checks: '-*,clang-diagnostic-*,llvm-*,misc-*,-misc-unused-parameters,-misc-non-private-member-variables-in-classes,-readability-identifier-naming'
+Checks: '-*,clang-diagnostic-*,llvm-*,misc-*,-misc-unused-parameters,-misc-non-private-member-variables-in-classes,-readability-identifier-naming,-misc-no-recursion'
 # Note that the readability-identifier-naming check is disabled, there are too
 # many violations in the codebase and they create too much noise in clang-tidy
 # results.