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. _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
