[Mlir-commits] [mlir] ab6502e - Enable a few clang-tidy checks in MLIR

[Mehdi Amini]

Author: Mehdi Amini
Date: 2022-01-02T01:05:06Z
New Revision: ab6502ea67378d972db8179a48936b102c442105

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

LOG: Enable a few clang-tidy checks in MLIR

The dry-run of clang-tidy on the codebase with these enable were
well receive, and the codebase is "clean" (or almost) with respect
to these right now.

Added: 
    

Modified: 
    mlir/.clang-tidy

Removed: 
    


################################################################################
diff  --git a/mlir/.clang-tidy b/mlir/.clang-tidy
index acd9361b526d6..88da7aba0a111 100644
--- a/mlir/.clang-tidy
+++ b/mlir/.clang-tidy
@@ -1,4 +1,54 @@
 InheritParentConfig: true
+Checks: >
+        bugprone-argument-comment,
+        bugprone-assert-side-effect,
+        bugprone-branch-clone,
+        bugprone-copy-constructor-init,
+        bugprone-dangling-handle,
+        bugprone-dynamic-static-initializers,
+        bugprone-macro-parentheses,
+        bugprone-macro-repeated-side-effects,
+        bugprone-misplaced-widening-cast,
+        bugprone-move-forwarding-reference,
+        bugprone-multiple-statement-macro,
+        bugprone-suspicious-semicolon,
+        bugprone-swapped-arguments,
+        bugprone-terminating-continue,
+        bugprone-unused-raii,
+        bugprone-unused-return-value,
+        misc-redundant-expression,
+        misc-static-assert,
+        misc-unused-using-decls,
+        modernize-use-bool-literals,
+        modernize-loop-convert,
+        modernize-make-unique,
+        modernize-raw-string-literal,
+        modernize-use-equals-default,
+        modernize-use-default-member-init,
+        modernize-use-emplace,
+        modernize-use-nullptr,
+        modernize-use-override,
+        modernize-use-using,
+        performance-for-range-copy,
+        performance-implicit-conversion-in-loop,
+        performance-inefficient-algorithm,
+        performance-inefficient-vector-operation,
+        performance-move-const-arg,
+        performance-no-automatic-move,
+        performance-trivially-destructible,
+        performance-unnecessary-copy-initialization,
+        performance-unnecessary-value-param,
+        readability-avoid-const-params-in-decls,
+        readability-const-return-type,
+        readability-container-size-empty,
+        readability-inconsistent-declaration-parameter-name,
+        readability-misleading-indentation,
+        readability-redundant-control-flow,
+        readability-simplify-boolean-expr,
+        readability-simplify-subscript-expr,
+        readability-use-anyofallof
+
+
 CheckOptions:
   - key:             readability-identifier-naming.MemberCase
     value:           camelBack