diff options
author | Georgios Pinitas <georgios.pinitas@arm.com> | 2017-09-27 11:20:48 +0100 |
---|---|---|
committer | Anthony Barbier <anthony.barbier@arm.com> | 2018-11-02 16:35:24 +0000 |
commit | 30f0215ba247c05a81e91ed420f14b686a3528f3 (patch) | |
tree | 152ad7662b793ccc04a8804009dd5b5887e7de7d /scripts | |
parent | 74180bb942f681c535e2e1c483d4c8a8edfdb4a7 (diff) | |
download | ComputeLibrary-30f0215ba247c05a81e91ed420f14b686a3528f3.tar.gz |
COMPMID-417: Fix clang-tidy failures for 32-bit runs
Change-Id: I2fbb6dda1c281627a4d64dce3b4c4d2ebaa8d022
Reviewed-on: http://mpd-gerrit.cambridge.arm.com/89289
Reviewed-by: Pablo Tello <pablo.tello@arm.com>
Tested-by: Kaizen <jeremy.johnson+kaizengerrit@arm.com>
Reviewed-by: Anthony Barbier <anthony.barbier@arm.com>
Diffstat (limited to 'scripts')
-rwxr-xr-x | scripts/clang_tidy_rules.py | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/scripts/clang_tidy_rules.py b/scripts/clang_tidy_rules.py index 900413c90b..378e18eeff 100755 --- a/scripts/clang_tidy_rules.py +++ b/scripts/clang_tidy_rules.py @@ -14,8 +14,8 @@ def get_list_flags( filename, arch): if "tests/validation_old" in filename: flags.append("-DBOOST") flags.append("-DARM_COMPUTE_CPP_SCHEDULER=1") + flags.append("-DARM_COMPUTE_CL") if arch == "aarch64": - flags.append("-DARM_COMPUTE_CL") flags.append("-DARM_COMPUTE_ENABLE_FP16") return flags @@ -70,6 +70,7 @@ def filter_clang_tidy_lines( lines ): ("Error.cpp" in line and "do not define a C-style variadic function" 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 ("PMUCounter.cpp" in line and "consider replacing 'long long' with 'int64'" in line) or ("Validation.cpp" in line and "parameter 'classified_labels' is unused" in line) or ("Validation.cpp" in line and "parameter 'expected_labels' is unused" in line) or @@ -82,6 +83,7 @@ def filter_clang_tidy_lines( lines ): ("NEGEMMMatrixMultiplyKernel.cpp" in line and "do not use C-style cast to convert between unrelated types" in line) or ("NEPoolingLayerKernel.cpp" in line and "do not use C-style cast to convert between unrelated types" in line) or ("NESoftmaxLayerKernel.cpp" in line and "do not use C-style cast to convert between unrelated types" in line) or + ("GraphUtils.cpp" in line and "consider replacing 'unsigned long' with 'uint32'" in line) or ("GraphUtils.cpp" in line and "consider replacing 'unsigned long' with 'uint64'" in line) or ("parameter 'memory_manager' is unused" in line) or ("parameter 'memory_manager' is copied for each invocation but only used as a const reference" in line) or @@ -110,6 +112,12 @@ def filter_clang_tidy_lines( lines ): out.append(line) print_context=True + elif (("CLMinMaxLocationKernel.cpp" in line and "'?' condition is false" in line) or + ("CLMinMaxLocationKernel.cpp" in line and "Assuming the condition is false" in line) or + ("CLMinMaxLocationKernel.cpp" in line and "Assuming pointer value is null" in line) or + ("CLMinMaxLocationKernel.cpp" in line and "Forming reference to null pointer" in line)): + print_context=False + continue elif print_context: out.append(line) |