From: Aaron Ballman Date: Thu, 27 May 2021 14:37:33 +0000 (-0400) Subject: Disable misc-no-recursion checking in Clang X-Git-Tag: llvmorg-14-init~5440 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=767d34e3bdddef6c1871006dd0a2d06a4e1bcd5d;p=platform%2Fupstream%2Fllvm.git 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. --- diff --git a/clang/.clang-tidy b/clang/.clang-tidy index 5b425a7..f517e92 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.