Skip to content

GitLab

Commit 1d6e4134 authored by Paul's avatar Paul
Browse files

Upgrade cppcheck version

parent 799e8997
Hide whitespace changes
Inline Side-by-side
Showing with 42 additions and 5 deletions
CMakeLists.txt
View file @ 1d6e4134
...@@ -77,7 +77,7 @@ rocm_enable_clang_tidy( ...@@ -77,7 +77,7 @@ rocm_enable_clang_tidy(
) )
include(ROCMCppCheck) include(ROCMCppCheck)
rocm_enable_cppcheck( rocm_enable_cppcheck(
CHECKS CHECKS
warning warning
style style
performance performance
...@@ -89,7 +89,10 @@ rocm_enable_cppcheck( ...@@ -89,7 +89,10 @@ rocm_enable_cppcheck(
noExplicitConstructor noExplicitConstructor
passedByValue passedByValue
unusedStructMember unusedStructMember
definePrefix:*test/include/test.hpp
FORCE FORCE
RULE_FILE
${CMAKE_CURRENT_SOURCE_DIR}/cppcheck.rules
SOURCES SOURCES
src/ src/
test/ test/
......
Jenkinsfile
View file @ 1d6e4134
...@@ -69,7 +69,6 @@ def rocmnode(body) { ...@@ -69,7 +69,6 @@ def rocmnode(body) {
rocmtest tidy: rocmnode('rocmtest') { cmake_build -> rocmtest tidy: rocmnode('rocmtest') { cmake_build ->
stage('Clang Tidy') { stage('Clang Tidy') {
sh ''' sh '''
sed -i 's/ #/ /g' .clang-tidy
rm -rf build rm -rf build
mkdir build mkdir build
cd build cd build
......
cppcheck.rules 0 → 100644
View file @ 1d6e4134
<?xml version="1.0"?>
<rule>
<tokenlist>define</tokenlist>
<pattern>define [0-9A-Z_^a-z]*[a-z]</pattern>
<message>
<id>defineUpperCase</id>
<severity>style</severity>
<summary>Macros must be uppercase</summary>
</message>
</rule>
<rule>
<tokenlist>define</tokenlist>
<pattern>define (MIGRAP|[^H]{6})[^H][^_]</pattern>
<message>
<id>definePrefix</id>
<severity>style</severity>
<summary>Macros must be prefixed with MIGRAPH_</summary>
</message>
</rule>
<rule>
<pattern>memcpy \(</pattern>
<message>
<id>useStlAlgorithms</id>
<severity>style</severity>
<summary>Use std::copy instead of memcpy</summary>
</message>
</rule>
<rule>
<pattern>memset \(</pattern>
<message>
<id>useStlAlgorithms</id>
<severity>style</severity>
<summary>Use std::fill instead of memset</summary>
</message>
</rule>
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment