diff options
| author | Michalis Spyrou <michalis.spyrou@arm.com> | 2019-05-01 13:03:59 +0100 |
|---|---|---|
| committer | Michalis Spyrou <michalis.spyrou@arm.com> | 2019-05-10 13:30:46 +0000 |
| commit | bcfd09a14a61bb8457555c61e7c5697b1f901ddb (patch) | |
| tree | 30754b3eaad2464c787d74b304bcd7b135dd4178 /scripts | |
| parent | 46a49a0a8206f0efa7afd514940e180a88ffd732 (diff) | |
| download | ComputeLibrary-bcfd09a14a61bb8457555c61e7c5697b1f901ddb.tar.gz | |
COMPMID-2177 Fix clang warnings
Change-Id: I78039db8c58d7b14a042c41e54c25fb9cb509bf7
Signed-off-by: Michalis Spyrou <michalis.spyrou@arm.com>
Reviewed-on: https://review.mlplatform.org/c/1092
Reviewed-by: VidhyaSudhan Loganathan <vidhyasudhan.loganathan@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Diffstat (limited to 'scripts')
| -rwxr-xr-x | scripts/clang_tidy_rules.py | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/scripts/clang_tidy_rules.py b/scripts/clang_tidy_rules.py index d35273d545..cba226f5d7 100755 --- a/scripts/clang_tidy_rules.py +++ b/scripts/clang_tidy_rules.py @@ -42,6 +42,9 @@ def filter_clang_tidy_lines( lines ): if "/convolution/" in line: continue + if "/validate_examples/" in line: + continue + if "error:" in line: if (("Utils.cpp" in line and "'arm_compute_version.embed' file not found" in line) or ("arm_fp16.h" in line) or @@ -62,11 +65,14 @@ def filter_clang_tidy_lines( lines ): elif "warning:" in line: if ("uninitialized record type: '__ret'" in line or "local variable '__bound_functor' is still referred to by the global variable '__once_callable'" in line or + "assigning newly created 'gsl::owner<>'" in line or (any(f in line for f in ["Error.cpp","Error.h"]) and "thrown exception type is not nothrow copy constructible" in line) or (any(f in line for f in ["Error.cpp","Error.h"]) and "uninitialized record type: 'args'" in line) or (any(f in line for f in ["Error.cpp","Error.h"]) and "do not call c-style vararg functions" in line) or (any(f in line for f in ["Error.cpp","Error.h"]) and "do not define a C-style variadic function" in line) or ("TensorAllocator.cpp" in line and "warning: pointer parameter 'ptr' can be pointer to const" in line) or + ("TensorAllocator.cpp" in line and "warning: do not declare C-style arrays" in line) or + ("RawTensor.cpp" in line and "warning: pointer parameter 'ptr' can be pointer to const" in line) or ("NEMinMaxLocationKernel.cpp" in line and "move constructors should be marked noexcept" in line) or ("NEMinMaxLocationKernel.cpp" in line and "move assignment operators should be marked noexcept" in line) or ("CLMinMaxLocationKernel.cpp" in line and "Forming reference to null pointer" in line) or @@ -95,6 +101,8 @@ def filter_clang_tidy_lines( lines ): ("CPUUtils.cpp" in line and "consider replacing 'unsigned long' with 'uint64'" in line) or ("CPUUtils.cpp" in line and "parameter 'cpusv' is unused" in line) or ("CPUUtils.cpp" in line and "warning: uninitialized record type" in line) or + ("GCKernelLibrary.cpp" in line and "warning: do not declare C-style arrays" in line) or + ("Utils.h" in line and "warning: Use of zero-allocated memory" in line) or "3rdparty" in line): print_context=False continue |