Michalis Spyrou | 92ad0ca | 2019-12-17 11:43:32 +0000 | [diff] [blame] | 1 | #!/usr/bin/env python3 |
Anthony Barbier | 3dfbdeb | 2017-09-12 16:04:45 +0100 | [diff] [blame] | 2 | import os |
| 3 | import re |
| 4 | import sys |
| 5 | |
| 6 | def get_list_includes(): |
Georgios Pinitas | 51826a4 | 2019-08-02 18:01:21 +0100 | [diff] [blame] | 7 | return "arm_compute/core/NEON/kernels/assembly " \ |
| 8 | "arm_compute/core/NEON/kernels/convolution/common " \ |
| 9 | "arm_compute/core/NEON/kernels/convolution/depthwise " \ |
| 10 | "arm_compute/core/NEON/kernels/convolution/winograd " \ |
Michele Di Giorgio | 6ad60af | 2020-06-09 14:52:15 +0100 | [diff] [blame] | 11 | "src/core/NEON/kernels/assembly " \ |
| 12 | "src/core/NEON/kernels/convolution/winograd " \ |
Georgios Pinitas | 51826a4 | 2019-08-02 18:01:21 +0100 | [diff] [blame] | 13 | "include/linux include " \ |
| 14 | ". " \ |
| 15 | "3rdparty/include kernels".split() |
Anthony Barbier | 3dfbdeb | 2017-09-12 16:04:45 +0100 | [diff] [blame] | 16 | |
| 17 | def get_list_flags( filename, arch): |
| 18 | assert arch in ["armv7", "aarch64"] |
| 19 | flags = ["-std=c++11"] |
Anthony Barbier | 3dfbdeb | 2017-09-12 16:04:45 +0100 | [diff] [blame] | 20 | flags.append("-DARM_COMPUTE_CPP_SCHEDULER=1") |
Georgios Pinitas | 30f0215 | 2017-09-27 11:20:48 +0100 | [diff] [blame] | 21 | flags.append("-DARM_COMPUTE_CL") |
Anthony Barbier | 7068f99 | 2017-10-26 15:23:08 +0100 | [diff] [blame] | 22 | flags.append("-DARM_COMPUTE_GC") |
Anthony Barbier | 3dfbdeb | 2017-09-12 16:04:45 +0100 | [diff] [blame] | 23 | if arch == "aarch64": |
Ioan-Cristian Szabo | 33fd07b | 2017-10-26 15:42:24 +0100 | [diff] [blame] | 24 | flags.append("-DARM_COMPUTE_AARCH64_V8_2") |
Anthony Barbier | 3dfbdeb | 2017-09-12 16:04:45 +0100 | [diff] [blame] | 25 | return flags |
| 26 | |
| 27 | def filter_files( list_files ): |
| 28 | to_check = [] |
| 29 | for f in list_files: |
| 30 | if os.path.splitext(f)[1] != ".cpp": |
| 31 | continue |
Anthony Barbier | 3dfbdeb | 2017-09-12 16:04:45 +0100 | [diff] [blame] | 32 | # Skip OMPScheduler as it causes problems in clang |
Anthony Barbier | b467021 | 2018-05-18 16:55:39 +0100 | [diff] [blame] | 33 | if (("OMPScheduler.cpp" in f) or |
| 34 | ("CLTracePoint.cpp" in f) or |
| 35 | ("NETracePoint.cpp" in f) or |
| 36 | ("TracePoint.cpp" in f)): |
Anthony Barbier | 3dfbdeb | 2017-09-12 16:04:45 +0100 | [diff] [blame] | 37 | continue |
| 38 | to_check.append(f) |
| 39 | return to_check |
| 40 | |
| 41 | def filter_clang_tidy_lines( lines ): |
| 42 | out = [] |
| 43 | print_context=False |
| 44 | for i in range(0, len(lines)): |
| 45 | line = lines[i] |
| 46 | |
Pablo Tello | eb82fd2 | 2018-02-23 13:43:50 +0000 | [diff] [blame] | 47 | if "/arm_gemm/" in line: |
| 48 | continue |
| 49 | |
Georgios Pinitas | 4074c99 | 2018-01-30 18:13:46 +0000 | [diff] [blame] | 50 | if "/convolution/" in line: |
Pablo Tello | 8951933 | 2017-11-17 11:52:36 +0000 | [diff] [blame] | 51 | continue |
| 52 | |
Michalis Spyrou | bcfd09a | 2019-05-01 13:03:59 +0100 | [diff] [blame] | 53 | if "/validate_examples/" in line: |
| 54 | continue |
| 55 | |
Anthony Barbier | 3dfbdeb | 2017-09-12 16:04:45 +0100 | [diff] [blame] | 56 | if "error:" in line: |
Georgios Pinitas | 51545e4 | 2020-02-11 15:29:01 +0000 | [diff] [blame] | 57 | if (("Version.cpp" in line and "arm_compute_version.embed" in line and "file not found" in line) or |
Anthony Barbier | 3dfbdeb | 2017-09-12 16:04:45 +0100 | [diff] [blame] | 58 | ("arm_fp16.h" in line) or |
| 59 | ("omp.h" in line) or |
Moritz Pflanzer | beabe3b | 2017-08-31 14:56:32 +0100 | [diff] [blame] | 60 | ("cast from pointer to smaller type 'uintptr_t' (aka 'unsigned int') loses information" in line) or |
Anthony Barbier | b6eb353 | 2018-08-08 13:20:04 +0100 | [diff] [blame] | 61 | ("cast from pointer to smaller type 'cl_context_properties' (aka 'int') loses information" in line) or |
Moritz Pflanzer | beabe3b | 2017-08-31 14:56:32 +0100 | [diff] [blame] | 62 | ("cast from pointer to smaller type 'std::uintptr_t' (aka 'unsigned int') loses information" in line) or |
Anthony Barbier | 3dfbdeb | 2017-09-12 16:04:45 +0100 | [diff] [blame] | 63 | ("NEMath.inl" in line and "statement expression not allowed at file scope" in line) or |
Anthony Barbier | 2a07e18 | 2017-08-04 18:20:27 +0100 | [diff] [blame] | 64 | ("Utils.h" in line and "no member named 'unmap' in 'arm_compute::Tensor'" in line) or |
| 65 | ("Utils.h" in line and "no member named 'map' in 'arm_compute::Tensor'" in line) or |
Pablo Tello | 7fad9b1 | 2018-03-14 17:55:27 +0000 | [diff] [blame] | 66 | ("CPUUtils.cpp" in line and "'asm/hwcap.h' file not found" in line) or |
Manuel Bottini | 569b7e9 | 2020-03-20 14:37:40 +0000 | [diff] [blame] | 67 | "3rdparty" in line or |
| 68 | ("'arm_compute_version.embed' file not found" in line) ): |
Anthony Barbier | 3dfbdeb | 2017-09-12 16:04:45 +0100 | [diff] [blame] | 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 |
Michalis Spyrou | 2388de1 | 2019-05-13 15:51:05 +0100 | [diff] [blame] | 77 | "assigning newly created 'gsl::owner<>'" in line or |
Georgios Pinitas | b8d5b95 | 2019-05-16 14:13:03 +0100 | [diff] [blame] | 78 | "calling legacy resource function without passing a 'gsl::owner<>'" in line or |
Michalis Spyrou | 2388de1 | 2019-05-13 15:51:05 +0100 | [diff] [blame] | 79 | "deleting a pointer through a type that is not marked 'gsl::owner<>'" in line or |
Georgios Pinitas | 3faea25 | 2017-10-30 14:13:50 +0000 | [diff] [blame] | 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 |
Georgios Pinitas | 84b51ad | 2017-11-07 13:24:57 +0000 | [diff] [blame] | 84 | ("TensorAllocator.cpp" in line and "warning: pointer parameter 'ptr' can be pointer to const" in line) or |
Michalis Spyrou | bcfd09a | 2019-05-01 13:03:59 +0100 | [diff] [blame] | 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 |
Michalis Spyrou | 7dbfc41 | 2019-05-13 13:21:28 +0100 | [diff] [blame] | 87 | ("RawTensor.cpp" in line and "warning: do not declare C-style arrays" in line) or |
| 88 | ("GCBufferAllocator.cpp" in line and "warning: initializing non-owner" in line) or |
Anthony Barbier | 3dfbdeb | 2017-09-12 16:04:45 +0100 | [diff] [blame] | 89 | ("NEMinMaxLocationKernel.cpp" in line and "move constructors should be marked noexcept" in line) or |
| 90 | ("NEMinMaxLocationKernel.cpp" in line and "move assignment operators should be marked noexcept" in line) or |
Georgios Pinitas | 30f0215 | 2017-09-27 11:20:48 +0100 | [diff] [blame] | 91 | ("CLMinMaxLocationKernel.cpp" in line and "Forming reference to null pointer" in line) or |
Anthony Barbier | 3dfbdeb | 2017-09-12 16:04:45 +0100 | [diff] [blame] | 92 | ("PMUCounter.cpp" in line and "consider replacing 'long long' with 'int64'" in line) or |
| 93 | ("Validation.cpp" in line and "parameter 'classified_labels' is unused" in line) or |
| 94 | ("Validation.cpp" in line and "parameter 'expected_labels' is unused" in line) or |
| 95 | ("Reference.cpp" in line and "parameter 'rois' is unused" in line) or |
| 96 | ("Reference.cpp" in line and "parameter 'shapes' is unused" in line) or |
| 97 | ("Reference.cpp" in line and re.search(r"parameter '[^']+' is unused", line)) or |
| 98 | ("ReferenceCPP.cpp" in line and "parameter 'rois' is unused" in line) or |
| 99 | ("ReferenceCPP.cpp" in line and "parameter 'srcs' is unused" in line) or |
| 100 | ("ReferenceCPP.cpp" in line and re.search(r"parameter '[^']+' is unused", line)) or |
| 101 | ("NEGEMMMatrixMultiplyKernel.cpp" in line and "do not use C-style cast to convert between unrelated types" in line) or |
| 102 | ("NEPoolingLayerKernel.cpp" in line and "do not use C-style cast to convert between unrelated types" in line) or |
Diego Lopez Recas | 35ceeb2 | 2017-12-04 18:56:10 +0000 | [diff] [blame] | 103 | ("NESoftmaxLayerKernel.cpp" in line and "macro argument should be enclosed in parentheses" in line) or |
Georgios Pinitas | 30f0215 | 2017-09-27 11:20:48 +0100 | [diff] [blame] | 104 | ("GraphUtils.cpp" in line and "consider replacing 'unsigned long' with 'uint32'" in line) or |
Anthony Barbier | 2a07e18 | 2017-08-04 18:20:27 +0100 | [diff] [blame] | 105 | ("GraphUtils.cpp" in line and "consider replacing 'unsigned long' with 'uint64'" in line) or |
Georgios Pinitas | 6f669f0 | 2017-09-26 12:32:57 +0100 | [diff] [blame] | 106 | ("ConvolutionLayer.cpp" in line and "move assignment operators should be marked noexcept" in line) or |
| 107 | ("ConvolutionLayer.cpp" in line and "move constructors should be marked noexcept" in line) or |
Georgios Pinitas | baf174e | 2017-09-08 19:47:30 +0100 | [diff] [blame] | 108 | ("parameter 'memory_manager' is unused" in line) or |
Georgios Pinitas | 658039b | 2017-09-15 16:30:50 +0100 | [diff] [blame] | 109 | ("parameter 'memory_manager' is copied for each invocation but only used as a const reference" in line) or |
Pablo Tello | daaa1fa | 2017-10-25 11:40:50 +0100 | [diff] [blame] | 110 | ("DeconvolutionLayer.cpp" in line and "casting (double + 0.5) to integer leads to incorrect rounding; consider using lround" in line) or |
Pablo Tello | 3d4968a | 2017-12-04 15:03:35 +0000 | [diff] [blame] | 111 | ("NEWinogradLayerKernel.cpp" in line and "use '= default' to define a trivial destructor" in line) or |
Pablo Tello | eb82fd2 | 2018-02-23 13:43:50 +0000 | [diff] [blame] | 112 | ("NEGEMMLowpMatrixMultiplyCore.cpp" in line and "constructor does not initialize these fields" in line) or |
| 113 | ("NEGEMMLowpAssemblyMatrixMultiplyCore" in line and "constructor does not initialize these fields" in line) or |
Sang-Hoon Park | 68dd25f | 2020-10-19 16:00:11 +0100 | [diff] [blame] | 114 | ("NEDepthwiseConvolutionLayerNativeKernel" in line and re.search(r"parameter '[^']+' is unused", line)) or |
| 115 | ("NEDepthwiseConvolutionAssemblyDispatch" in line and re.search(r"parameter '[^']+' is unused", line)) or |
Pablo Tello | 7fad9b1 | 2018-03-14 17:55:27 +0000 | [diff] [blame] | 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 |
Michalis Spyrou | 8e5174c | 2018-12-04 11:43:23 +0000 | [diff] [blame] | 118 | ("CPUUtils.cpp" in line and "warning: uninitialized record type" in line) or |
Michalis Spyrou | bcfd09a | 2019-05-01 13:03:59 +0100 | [diff] [blame] | 119 | ("GCKernelLibrary.cpp" in line and "warning: do not declare C-style arrays" in line) or |
| 120 | ("Utils.h" in line and "warning: Use of zero-allocated memory" in line) or |
Sang-Hoon Park | 68dd25f | 2020-10-19 16:00:11 +0100 | [diff] [blame] | 121 | ("NEDepthwiseConvolutionLayerNativeKernel.cpp" in line and "misc-non-private-member-variables-in-classes" in line) or # This is to prevent false positive, should be reassessed with the newer clang-tidy |
| 122 | ("NEDepthwiseConvolutionLayerNativeKernel.cpp" in line and "cppcoreguidelines-pro-type-member-init" in line) or # This is to prevent false positive, should be reassessed with the newer clang-tidy |
Anthony Barbier | 3dfbdeb | 2017-09-12 16:04:45 +0100 | [diff] [blame] | 123 | "3rdparty" in line): |
| 124 | print_context=False |
| 125 | continue |
| 126 | |
| 127 | if "do not use C-style cast to convert between unrelated types" in line: |
| 128 | if i + 1 < len(lines) and "vgetq_lane_f16" in lines[i + 1]: |
| 129 | print_context=False |
| 130 | continue |
| 131 | |
| 132 | if "use 'using' instead of 'typedef'" in line: |
| 133 | if i + 1 < len(lines) and "BOOST_FIXTURE_TEST_SUITE" in lines[i + 1]: |
| 134 | print_context=False |
| 135 | continue |
| 136 | |
| 137 | if "do not call c-style vararg functions" in line: |
| 138 | if (i + 1 < len(lines) and |
| 139 | ("BOOST_TEST" in lines[i + 1] or |
| 140 | "BOOST_FAIL" in lines[i + 1] or |
| 141 | "BOOST_CHECK_THROW" in lines[i + 1] or |
Michalis Spyrou | 7c60c99 | 2019-10-10 14:33:47 +0100 | [diff] [blame] | 142 | "ARM_COMPUTE_ERROR_VAR" in lines[i + 1] or |
Georgios Pinitas | 3faea25 | 2017-10-30 14:13:50 +0000 | [diff] [blame] | 143 | "ARM_COMPUTE_RETURN_ON" in lines[i + 1] or |
Anthony Barbier | 3dfbdeb | 2017-09-12 16:04:45 +0100 | [diff] [blame] | 144 | "syscall" in lines[i + 1])): |
| 145 | print_context=False |
| 146 | continue |
| 147 | |
| 148 | out.append(line) |
| 149 | print_context=True |
Georgios Pinitas | 30f0215 | 2017-09-27 11:20:48 +0100 | [diff] [blame] | 150 | elif (("CLMinMaxLocationKernel.cpp" in line and "'?' condition is false" in line) or |
| 151 | ("CLMinMaxLocationKernel.cpp" in line and "Assuming the condition is false" in line) or |
| 152 | ("CLMinMaxLocationKernel.cpp" in line and "Assuming pointer value is null" in line) or |
| 153 | ("CLMinMaxLocationKernel.cpp" in line and "Forming reference to null pointer" in line)): |
| 154 | print_context=False |
| 155 | continue |
Anthony Barbier | 3dfbdeb | 2017-09-12 16:04:45 +0100 | [diff] [blame] | 156 | elif print_context: |
| 157 | out.append(line) |
| 158 | |
| 159 | return out |
| 160 | |
| 161 | if __name__ == "__main__": |
| 162 | if len(sys.argv) != 2: |
| 163 | print("usage: {} CLANG-TIDY_OUTPUT_FILE".format(sys.argv[0])) |
| 164 | sys.exit(1) |
| 165 | |
| 166 | errors = [] |
| 167 | with open(sys.argv[1], mode="r") as clang_tidy_file: |
| 168 | lines = clang_tidy_file.readlines() |
| 169 | errors = filter_clang_tidy_lines(lines) |
Michalis Spyrou | 92ad0ca | 2019-12-17 11:43:32 +0000 | [diff] [blame] | 170 | print("\n".join(errors)) |
Anthony Barbier | 3dfbdeb | 2017-09-12 16:04:45 +0100 | [diff] [blame] | 171 | |
| 172 | sys.exit(0 if len(errors) == 0 else 1) |