From 39b61b59286bec0c94d009c04281f0c2e21ea7b4 Mon Sep 17 00:00:00 2001 From: Mateusz Jablonski Date: Mon, 1 Jun 2020 18:38:23 +0200 Subject: [PATCH] Add .clang-tidy file for level zero and shared tests add some extra checks for clang-tidy in tests: - clang-analyzer-core.StackAddressEscape - clang-analyzer-core.uninitialized.Assign - clang-analyzer-core.NonNullParamChecker - readability-identifier-naming.StructMemberCase Change-Id: Ib8f82d5555f65635f2e7a6f82287bd480a508a7f Signed-off-by: Mateusz Jablonski --- .../core/test}/.clang-tidy | 6 ++-- level_zero/tools/test/.clang-tidy | 34 +++++++++++++++++++ opencl/test/.clang-tidy | 34 +++++++++++++++++++ shared/test/.clang-tidy | 34 +++++++++++++++++++ 4 files changed, 106 insertions(+), 2 deletions(-) rename {opencl/test/unit_test => level_zero/core/test}/.clang-tidy (73%) create mode 100644 level_zero/tools/test/.clang-tidy create mode 100644 opencl/test/.clang-tidy create mode 100644 shared/test/.clang-tidy diff --git a/opencl/test/unit_test/.clang-tidy b/level_zero/core/test/.clang-tidy similarity index 73% rename from opencl/test/unit_test/.clang-tidy rename to level_zero/core/test/.clang-tidy index 4365adcff2..ba07f3f657 100644 --- a/opencl/test/unit_test/.clang-tidy +++ b/level_zero/core/test/.clang-tidy @@ -1,7 +1,7 @@ --- -Checks: 'clang-diagnostic-*,clang-analyzer-*,google-default-arguments,modernize-use-override,modernize-use-default-member-init,-clang-analyzer-alpha*,readability-identifier-naming,-clang-analyzer-core.StackAddressEscape,-clang-analyzer-optin.performance.Padding,-clang-analyzer-security.insecureAPI.strcpy,-clang-analyzer-cplusplus.NewDeleteLeaks,-clang-analyzer-core.CallAndMessage,-clang-analyzer-core.uninitialized.Assign,-clang-analyzer-unix.MismatchedDeallocator,-clang-analyzer-core.NonNullParamChecker,-clang-analyzer-core.NullDereference,-clang-analyzer-cplusplus.NewDelete,-clang-analyzer-optin.cplusplus.VirtualCall' +Checks: 'clang-diagnostic-*,clang-analyzer-*,google-default-arguments,modernize-use-override,modernize-use-default-member-init,-clang-analyzer-alpha*,readability-identifier-naming,-clang-analyzer-optin.performance.Padding,-clang-analyzer-security.insecureAPI.strcpy,-clang-analyzer-cplusplus.NewDeleteLeaks,-clang-analyzer-core.CallAndMessage,-clang-analyzer-unix.MismatchedDeallocator,-clang-analyzer-core.NullDereference,-clang-analyzer-cplusplus.NewDelete,-clang-analyzer-optin.cplusplus.VirtualCall' # WarningsAsErrors: '.*' -HeaderFilterRegex: '/runtime/|/core/|/offline_compiler/' +HeaderFilterRegex: '*' AnalyzeTemporaryDtors: false CheckOptions: - key: google-readability-braces-around-statements.ShortStatementLines @@ -14,6 +14,8 @@ CheckOptions: value: '2' - key: readability-identifier-naming.ParameterCase value: camelBack + - key: readability-identifier-naming.StructMemberCase + value: camelBack - key: modernize-loop-convert.MaxCopySize value: '16' - key: modernize-loop-convert.MinConfidence diff --git a/level_zero/tools/test/.clang-tidy b/level_zero/tools/test/.clang-tidy new file mode 100644 index 0000000000..ba07f3f657 --- /dev/null +++ b/level_zero/tools/test/.clang-tidy @@ -0,0 +1,34 @@ +--- +Checks: 'clang-diagnostic-*,clang-analyzer-*,google-default-arguments,modernize-use-override,modernize-use-default-member-init,-clang-analyzer-alpha*,readability-identifier-naming,-clang-analyzer-optin.performance.Padding,-clang-analyzer-security.insecureAPI.strcpy,-clang-analyzer-cplusplus.NewDeleteLeaks,-clang-analyzer-core.CallAndMessage,-clang-analyzer-unix.MismatchedDeallocator,-clang-analyzer-core.NullDereference,-clang-analyzer-cplusplus.NewDelete,-clang-analyzer-optin.cplusplus.VirtualCall' +# WarningsAsErrors: '.*' +HeaderFilterRegex: '*' +AnalyzeTemporaryDtors: false +CheckOptions: + - key: google-readability-braces-around-statements.ShortStatementLines + value: '1' + - key: google-readability-function-size.StatementThreshold + value: '800' + - key: google-readability-namespace-comments.ShortNamespaceLines + value: '10' + - key: google-readability-namespace-comments.SpacesBeforeComments + value: '2' + - key: readability-identifier-naming.ParameterCase + value: camelBack + - key: readability-identifier-naming.StructMemberCase + value: camelBack + - key: modernize-loop-convert.MaxCopySize + value: '16' + - key: modernize-loop-convert.MinConfidence + value: reasonable + - key: modernize-loop-convert.NamingStyle + value: CamelCase + - key: modernize-pass-by-value.IncludeStyle + value: llvm + - key: modernize-replace-auto-ptr.IncludeStyle + value: llvm + - key: modernize-use-nullptr.NullMacros + value: 'NULL' + - key: modernize-use-default-member-init.UseAssignment + value: '1' +... + diff --git a/opencl/test/.clang-tidy b/opencl/test/.clang-tidy new file mode 100644 index 0000000000..ba07f3f657 --- /dev/null +++ b/opencl/test/.clang-tidy @@ -0,0 +1,34 @@ +--- +Checks: 'clang-diagnostic-*,clang-analyzer-*,google-default-arguments,modernize-use-override,modernize-use-default-member-init,-clang-analyzer-alpha*,readability-identifier-naming,-clang-analyzer-optin.performance.Padding,-clang-analyzer-security.insecureAPI.strcpy,-clang-analyzer-cplusplus.NewDeleteLeaks,-clang-analyzer-core.CallAndMessage,-clang-analyzer-unix.MismatchedDeallocator,-clang-analyzer-core.NullDereference,-clang-analyzer-cplusplus.NewDelete,-clang-analyzer-optin.cplusplus.VirtualCall' +# WarningsAsErrors: '.*' +HeaderFilterRegex: '*' +AnalyzeTemporaryDtors: false +CheckOptions: + - key: google-readability-braces-around-statements.ShortStatementLines + value: '1' + - key: google-readability-function-size.StatementThreshold + value: '800' + - key: google-readability-namespace-comments.ShortNamespaceLines + value: '10' + - key: google-readability-namespace-comments.SpacesBeforeComments + value: '2' + - key: readability-identifier-naming.ParameterCase + value: camelBack + - key: readability-identifier-naming.StructMemberCase + value: camelBack + - key: modernize-loop-convert.MaxCopySize + value: '16' + - key: modernize-loop-convert.MinConfidence + value: reasonable + - key: modernize-loop-convert.NamingStyle + value: CamelCase + - key: modernize-pass-by-value.IncludeStyle + value: llvm + - key: modernize-replace-auto-ptr.IncludeStyle + value: llvm + - key: modernize-use-nullptr.NullMacros + value: 'NULL' + - key: modernize-use-default-member-init.UseAssignment + value: '1' +... + diff --git a/shared/test/.clang-tidy b/shared/test/.clang-tidy new file mode 100644 index 0000000000..ba07f3f657 --- /dev/null +++ b/shared/test/.clang-tidy @@ -0,0 +1,34 @@ +--- +Checks: 'clang-diagnostic-*,clang-analyzer-*,google-default-arguments,modernize-use-override,modernize-use-default-member-init,-clang-analyzer-alpha*,readability-identifier-naming,-clang-analyzer-optin.performance.Padding,-clang-analyzer-security.insecureAPI.strcpy,-clang-analyzer-cplusplus.NewDeleteLeaks,-clang-analyzer-core.CallAndMessage,-clang-analyzer-unix.MismatchedDeallocator,-clang-analyzer-core.NullDereference,-clang-analyzer-cplusplus.NewDelete,-clang-analyzer-optin.cplusplus.VirtualCall' +# WarningsAsErrors: '.*' +HeaderFilterRegex: '*' +AnalyzeTemporaryDtors: false +CheckOptions: + - key: google-readability-braces-around-statements.ShortStatementLines + value: '1' + - key: google-readability-function-size.StatementThreshold + value: '800' + - key: google-readability-namespace-comments.ShortNamespaceLines + value: '10' + - key: google-readability-namespace-comments.SpacesBeforeComments + value: '2' + - key: readability-identifier-naming.ParameterCase + value: camelBack + - key: readability-identifier-naming.StructMemberCase + value: camelBack + - key: modernize-loop-convert.MaxCopySize + value: '16' + - key: modernize-loop-convert.MinConfidence + value: reasonable + - key: modernize-loop-convert.NamingStyle + value: CamelCase + - key: modernize-pass-by-value.IncludeStyle + value: llvm + - key: modernize-replace-auto-ptr.IncludeStyle + value: llvm + - key: modernize-use-nullptr.NullMacros + value: 'NULL' + - key: modernize-use-default-member-init.UseAssignment + value: '1' +... +