Compute Library
 22.05
clang_tidy_rules Namespace Reference

Functions

def get_list_includes ()
 
def get_list_flags (filename, arch)
 
def filter_files (list_files)
 
def filter_clang_tidy_lines (lines)
 

Variables

list errors = []
 
 mode
 
 lines = clang_tidy_file.readlines()
 

Function Documentation

◆ filter_clang_tidy_lines()

def clang_tidy_rules.filter_clang_tidy_lines (   lines)

Definition at line 37 of file clang_tidy_rules.py.

References arm_compute::test::validation::reference.range().

37 def filter_clang_tidy_lines( lines ):
38  out = []
39  print_context=False
40  for i in range(0, len(lines)):
41  line = lines[i]
42 
43  if "/arm_conv/" in line:
44  continue
45 
46  if "/arm_gemm/" in line:
47  continue
48 
49  if "/convolution/" in line:
50  continue
51 
52  if "/validate_examples/" in line:
53  continue
54 
55  if "error:" in line:
56  if (("Version.cpp" in line and "arm_compute_version.embed" in line and "file not found" in line) or
57  ("arm_fp16.h" in line) or
58  ("omp.h" in line) or
59  ("cast from pointer to smaller type 'uintptr_t' (aka 'unsigned int') loses information" in line) or
60  ("cast from pointer to smaller type 'cl_context_properties' (aka 'int') loses information" in line) or
61  ("cast from pointer to smaller type 'std::uintptr_t' (aka 'unsigned int') loses information" in line) or
62  ("NEMath.inl" in line and "statement expression not allowed at file scope" in line) or
63  ("Utils.h" in line and "no member named 'unmap' in 'arm_compute::Tensor'" in line) or
64  ("Utils.h" in line and "no member named 'map' in 'arm_compute::Tensor'" in line) or
65  ("CPUUtils.cpp" in line and "'asm/hwcap.h' file not found" in line) or
66  ("CPUUtils.cpp" in line and "use of undeclared identifier 'HWCAP_SVE'" in line) or
67  ("sve" in line) or
68  ("'arm_compute_version.embed' file not found" in line) ):
69  print_context=False
70  continue
71 
72  out.append(line)
73  print_context=True
74  elif "warning:" in line:
75  if ("uninitialized record type: '__ret'" in line or
76  "local variable '__bound_functor' is still referred to by the global variable '__once_callable'" in line or
77  "assigning newly created 'gsl::owner<>'" in line or
78  "calling legacy resource function without passing a 'gsl::owner<>'" in line or
79  "deleting a pointer through a type that is not marked 'gsl::owner<>'" in line or
80  (any(f in line for f in ["Error.cpp","Error.h"]) and "thrown exception type is not nothrow copy constructible" in line) or
81  (any(f in line for f in ["Error.cpp","Error.h"]) and "uninitialized record type: 'args'" in line) or
82  (any(f in line for f in ["Error.cpp","Error.h"]) and "do not call c-style vararg functions" in line) or
83  (any(f in line for f in ["Error.cpp","Error.h"]) and "do not define a C-style variadic function" in line) or
84  ("TensorAllocator.cpp" in line and "warning: pointer parameter 'ptr' can be pointer to const" in line) or
85  ("TensorAllocator.cpp" in line and "warning: do not declare C-style arrays" in line) or
86  ("RawTensor.cpp" in line and "warning: pointer parameter 'ptr' can be pointer to const" in line) or
87  ("RawTensor.cpp" in line and "warning: do not declare C-style arrays" in line) or
88  ("NEMinMaxLocationKernel.cpp" in line and "move constructors should be marked noexcept" in line) or
89  ("NEMinMaxLocationKernel.cpp" in line and "move assignment operators should be marked noexcept" in line) or
90  ("CLMinMaxLocationKernel.cpp" in line and "Forming reference to null pointer" in line) or
91  ("PMUCounter.cpp" in line and "consider replacing 'long long' with 'int64'" in line) or
92  ("Validation.cpp" in line and "parameter 'classified_labels' is unused" in line) or
93  ("Validation.cpp" in line and "parameter 'expected_labels' is unused" in line) or
94  ("Reference.cpp" in line and "parameter 'rois' is unused" in line) or
95  ("Reference.cpp" in line and "parameter 'shapes' is unused" in line) or
96  ("Reference.cpp" in line and re.search(r"parameter '[^']+' is unused", line)) or
97  ("ReferenceCPP.cpp" in line and "parameter 'rois' is unused" in line) or
98  ("ReferenceCPP.cpp" in line and "parameter 'srcs' is unused" in line) or
99  ("ReferenceCPP.cpp" in line and re.search(r"parameter '[^']+' is unused", line)) or
100  ("NEGEMMMatrixMultiplyKernel.cpp" in line and "do not use C-style cast to convert between unrelated types" in line) or
101  ("NEPoolingLayerKernel.cpp" in line and "do not use C-style cast to convert between unrelated types" in line) or
102  ("NESoftmaxLayerKernel.cpp" in line and "macro argument should be enclosed in parentheses" in line) or
103  ("GraphUtils.cpp" in line and "consider replacing 'unsigned long' with 'uint32'" in line) or
104  ("GraphUtils.cpp" in line and "consider replacing 'unsigned long' with 'uint64'" in line) or
105  ("ConvolutionLayer.cpp" in line and "move assignment operators should be marked noexcept" in line) or
106  ("ConvolutionLayer.cpp" in line and "move constructors should be marked noexcept" in line) or
107  ("parameter 'memory_manager' is unused" in line) or
108  ("parameter 'memory_manager' is copied for each invocation but only used as a const reference" in line) or
109  ("DeconvolutionLayer.cpp" in line and "casting (double + 0.5) to integer leads to incorrect rounding; consider using lround" in line) or
110  ("NEWinogradLayerKernel.cpp" in line and "use '= default' to define a trivial destructor" in line) or
111  ("NEGEMMLowpMatrixMultiplyCore.cpp" in line and "constructor does not initialize these fields" in line) or
112  ("NEGEMMLowpAssemblyMatrixMultiplyCore" in line and "constructor does not initialize these fields" in line) or
113  ("CpuDepthwiseConv2dNativeKernel" in line and re.search(r"parameter '[^']+' is unused", line)) or
114  ("CpuDepthwiseConv2dAssemblyDispatch" in line and re.search(r"parameter '[^']+' is unused", line)) or
115  ("CpuDepthwiseConv2dAssemblyDispatch" in line and "modernize-use-equals-default" in line) or
116  ("CPUUtils.cpp" in line and "consider replacing 'unsigned long' with 'uint64'" in line) or
117  ("CPUUtils.cpp" in line and "parameter 'cpusv' is unused" in line) or
118  ("CPUUtils.cpp" in line and "warning: uninitialized record type" in line) or
119  ("Utils.h" in line and "warning: Use of zero-allocated memory" in line) or
120  ("sve" in line) or
121  ("CpuDepthwiseConv2dNativeKernel.cpp" in line and "misc-non-private-member-variables-in-classes" in line)): # This is to prevent false positive, should be reassessed with the newer clang-tidy
122  print_context=False
123  continue
124 
125  if "do not use C-style cast to convert between unrelated types" in line:
126  if i + 1 < len(lines) and "vgetq_lane_f16" in lines[i + 1]:
127  print_context=False
128  continue
129 
130  if "use 'using' instead of 'typedef'" in line:
131  if i + 1 < len(lines) and "BOOST_FIXTURE_TEST_SUITE" in lines[i + 1]:
132  print_context=False
133  continue
134 
135  if "do not call c-style vararg functions" in line:
136  if (i + 1 < len(lines) and
137  ("BOOST_TEST" in lines[i + 1] or
138  "BOOST_FAIL" in lines[i + 1] or
139  "BOOST_CHECK_THROW" in lines[i + 1] or
140  "ARM_COMPUTE_ERROR_VAR" in lines[i + 1] or
141  "ARM_COMPUTE_RETURN_ON" in lines[i + 1] or
142  "syscall" in lines[i + 1])):
143  print_context=False
144  continue
145 
146  out.append(line)
147  print_context=True
148  elif (("CLMinMaxLocationKernel.cpp" in line and "'?' condition is false" in line) or
149  ("CLMinMaxLocationKernel.cpp" in line and "Assuming the condition is false" in line) or
150  ("CLMinMaxLocationKernel.cpp" in line and "Assuming pointer value is null" in line) or
151  ("CLMinMaxLocationKernel.cpp" in line and "Forming reference to null pointer" in line)):
152  print_context=False
153  continue
154  elif print_context:
155  out.append(line)
156 
157  return out
158 
def filter_clang_tidy_lines(lines)
SimpleTensor< T > range(SimpleTensor< T > &dst, float start, const size_t num_of_elements, float step)
Definition: Range.cpp:50

◆ filter_files()

def clang_tidy_rules.filter_files (   list_files)

Definition at line 23 of file clang_tidy_rules.py.

23 def filter_files( list_files ):
24  to_check = []
25  for f in list_files:
26  if os.path.splitext(f)[1] != ".cpp":
27  continue
28  # Skip OMPScheduler as it causes problems in clang
29  if (("OMPScheduler.cpp" in f) or
30  ("CLTracePoint.cpp" in f) or
31  ("NETracePoint.cpp" in f) or
32  ("TracePoint.cpp" in f)):
33  continue
34  to_check.append(f)
35  return to_check
36 
def filter_files(list_files)

◆ get_list_flags()

def clang_tidy_rules.get_list_flags (   filename,
  arch 
)

Definition at line 13 of file clang_tidy_rules.py.

13 def get_list_flags( filename, arch):
14  assert arch in ["armv7", "aarch64"]
15  flags = ["-std=c++14"]
16  flags.append("-DARM_COMPUTE_CPP_SCHEDULER=1")
17  flags.append("-DARM_COMPUTE_CL")
18  flags.append("-DENABLE_EXPERIMENTAL_DYNAMIC_FUSION")
19  if arch == "aarch64":
20  flags.append("-DARM_COMPUTE_AARCH64_V8_2")
21  return flags
22 
def get_list_flags(filename, arch)

◆ get_list_includes()

def clang_tidy_rules.get_list_includes ( )

Definition at line 6 of file clang_tidy_rules.py.

7  return "src/cpu/kernels/assembly " \
8  "src/core/NEON/kernels/assembly " \
9  "src/core/NEON/kernels/convolution/winograd " \
10  "include/linux include " \
11  ". ".split()
12 

Variable Documentation

◆ errors

def errors = []

Definition at line 164 of file clang_tidy_rules.py.

◆ lines

lines = clang_tidy_file.readlines()

Definition at line 166 of file clang_tidy_rules.py.

◆ mode