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/NESobel5x5Kernel.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/NESobel5x5Kernel.cpp')
-rw-r--r-- | src/core/NEON/kernels/NESobel5x5Kernel.cpp | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/core/NEON/kernels/NESobel5x5Kernel.cpp b/src/core/NEON/kernels/NESobel5x5Kernel.cpp index 30e7817aa4..a92cfc2308 100644 --- a/src/core/NEON/kernels/NESobel5x5Kernel.cpp +++ b/src/core/NEON/kernels/NESobel5x5Kernel.cpp @@ -1,5 +1,5 @@ /* - * Copyright (c) 2016, 2017 ARM Limited. + * Copyright (c) 2016-2019 ARM Limited. * * SPDX-License-Identifier: MIT * @@ -120,7 +120,7 @@ void NESobel5x5HorKernel::run(const Window &window, const ThreadInfo &info) static const int16x8_t two = vdupq_n_s16(2); static const int16x8_t minustwo = vdupq_n_s16(-2); - execute_window_loop(window, [&](const Coordinates & id) + execute_window_loop(window, [&](const Coordinates &) { const uint8x16_t data = vld1q_u8(input.ptr()); @@ -154,7 +154,7 @@ void NESobel5x5HorKernel::run(const Window &window, const ThreadInfo &info) static const int16x8_t two = vdupq_n_s16(2); static const int16x8_t minustwo = vdupq_n_s16(-2); - execute_window_loop(window, [&](const Coordinates & id) + execute_window_loop(window, [&](const Coordinates &) { const uint8x16_t data = vld1q_u8(input.ptr()); @@ -180,7 +180,7 @@ void NESobel5x5HorKernel::run(const Window &window, const ThreadInfo &info) static const int16x8_t six = vdupq_n_s16(6); static const int16x8_t four = vdupq_n_s16(4); - execute_window_loop(window, [&](const Coordinates & id) + execute_window_loop(window, [&](const Coordinates &) { const uint8x16_t data = vld1q_u8(input.ptr()); @@ -211,7 +211,7 @@ NESobel5x5VertKernel::NESobel5x5VertKernel() BorderSize NESobel5x5VertKernel::border_size() const { - return BorderSize(2, 0); + return BorderSize{ 2, 0 }; } void NESobel5x5VertKernel::configure(ITensor *input_x, ITensor *input_y, ITensor *output_x, ITensor *output_y, bool border_undefined) @@ -312,7 +312,7 @@ void NESobel5x5VertKernel::run(const Window &window, const ThreadInfo &info) if(_run_sobel_x) { - execute_window_loop(window, [&](const Coordinates & id) + execute_window_loop(window, [&](const Coordinates &) { // Convert offset from uint8_t* to uint16_t* const size_t input_offset_high_s16 = input_x.offset() / 2; @@ -361,7 +361,7 @@ void NESobel5x5VertKernel::run(const Window &window, const ThreadInfo &info) if(_run_sobel_y) { - execute_window_loop(window, [&](const Coordinates & id) + execute_window_loop(window, [&](const Coordinates &) { // Convert offset from uint8_t* to uint16_t* const size_t input_offset_high_s16 = input_y.offset() / 2; |