diff options
author | Michalis Spyrou <michalis.spyrou@arm.com> | 2019-04-26 14:54:54 +0100 |
---|---|---|
committer | Michalis Spyrou <michalis.spyrou@arm.com> | 2019-05-01 10:06:58 +0000 |
commit | a4f378dcd39addd4a63db1c0848f2c120804f4eb (patch) | |
tree | 6fa8a0071bef32d2bdef0e5469678a7cfecea348 /src/core/NEON/kernels/NEMinMaxLocationKernel.cpp | |
parent | 8ec0bb6d9027bb7505d6fa0eada42a52c6e1073b (diff) | |
download | ComputeLibrary-a4f378dcd39addd4a63db1c0848f2c120804f4eb.tar.gz |
COMPMID-1995: Fix clang-tidy warnings
- Remove VirtualCall checks
- Fix some unused variables errors
- Use std::array insted of C style arrays
- Various fixes
Change-Id: Ife6170b7102de42b8f04e298dcf8476bf90779f0
Signed-off-by: Michalis Spyrou <michalis.spyrou@arm.com>
Reviewed-on: https://review.mlplatform.org/c/1049
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Michele Di Giorgio <michele.digiorgio@arm.com>
Diffstat (limited to 'src/core/NEON/kernels/NEMinMaxLocationKernel.cpp')
-rw-r--r-- | src/core/NEON/kernels/NEMinMaxLocationKernel.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/core/NEON/kernels/NEMinMaxLocationKernel.cpp b/src/core/NEON/kernels/NEMinMaxLocationKernel.cpp index befece2741..08b27e319e 100644 --- a/src/core/NEON/kernels/NEMinMaxLocationKernel.cpp +++ b/src/core/NEON/kernels/NEMinMaxLocationKernel.cpp @@ -1,5 +1,5 @@ /* - * Copyright (c) 2016-2018 ARM Limited. + * Copyright (c) 2016-2019 ARM Limited. * * SPDX-License-Identifier: MIT * @@ -152,7 +152,7 @@ void NEMinMaxKernel::minmax_U8(Window win) Iterator input(_input, win); - execute_window_loop(win, [&](const Coordinates & id) + execute_window_loop(win, [&](const Coordinates &) { int x = x_start; @@ -209,7 +209,7 @@ void NEMinMaxKernel::minmax_S16(Window win) Iterator input(_input, win); - execute_window_loop(win, [&](const Coordinates & id) + execute_window_loop(win, [&](const Coordinates &) { int x = x_start; const auto in_ptr = reinterpret_cast<const int16_t *>(input.ptr()); @@ -268,7 +268,7 @@ void NEMinMaxKernel::minmax_F32(Window win) Iterator input(_input, win); - execute_window_loop(win, [&](const Coordinates & id) + execute_window_loop(win, [&](const Coordinates &) { int x = x_start; const auto in_ptr = reinterpret_cast<const float *>(input.ptr()); @@ -323,11 +323,11 @@ bool NEMinMaxLocationKernel::is_parallelisable() const template <class T, std::size_t... N> struct NEMinMaxLocationKernel::create_func_table<T, utility::index_sequence<N...>> { - static const NEMinMaxLocationKernel::MinMaxLocFunction func_table[sizeof...(N)]; + static const std::array<NEMinMaxLocationKernel::MinMaxLocFunction, sizeof...(N)> func_table; }; template <class T, std::size_t... N> -const NEMinMaxLocationKernel::MinMaxLocFunction NEMinMaxLocationKernel::create_func_table<T, utility::index_sequence<N...>>::func_table[sizeof...(N)] = +const std::array<NEMinMaxLocationKernel::MinMaxLocFunction, sizeof...(N)> NEMinMaxLocationKernel::create_func_table<T, utility::index_sequence<N...>>::func_table { &NEMinMaxLocationKernel::minmax_loc<T, bool(N & 8), bool(N & 4), bool(N & 2), bool(N & 1)>... }; |