commit 8ef5706e520c25b451f1923f34ea8b7e96aa0742
author Michalis Spyrou <michalis.spyrou@arm.com> Tue Jan 14 12:15:48 2020 +0000
committer Michalis Spyrou <michalis.spyrou@arm.com> Tue Jan 14 13:26:01 2020 +0000
tree 3c43f256ef7b6b50545dcbe91989ba541c64a448
parent 959c26d0457deeebf7306b9e4317863f144415b5

COMPMID-3006 (Nightly) Build failure

Changed int type to uint32_t in order to fix
compiler warnings about signed overflow.

Change-Id: Ic250b159b9f8b1d4deebb5155923c9feb8d71c55
Signed-off-by: Michalis Spyrou <michalis.spyrou@arm.com>
Reviewed-on: https://review.mlplatform.org/c/2586
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Michele Di Giorgio <michele.digiorgio@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>

src/core/NEON/kernels/NEDirectConvolutionLayerKernel.cpp

diff --git a/src/core/NEON/kernels/NEDirectConvolutionLayerKernel.cpp b/src/core/NEON/kernels/NEDirectConvolutionLayerKernel.cpp
index 4a71c1e..fe3eb92 100644
--- a/src/core/NEON/kernels/NEDirectConvolutionLayerKernel.cpp
+++ b/src/core/NEON/kernels/NEDirectConvolutionLayerKernel.cpp
@@ -524,7 +524,7 @@
 class convolver_nhwc
 {
 public:
-    static void convolve(const Window &window, int kernel_size, unsigned int num_elems_read_per_iteration,
+    static void convolve(const Window &window, uint32_t kernel_size, unsigned int num_elems_read_per_iteration,
                          const ITensor *input, const ITensor *weights, ITensor *output, const PadStrideInfo &conv_info)
     {
         const int          input_width     = input->info()->dimension(0);
@@ -577,7 +577,7 @@
                 auto in_addr_base0 = in_ptr;
                 auto we_addr_base0 = k.ptr();
 
-                for(int z = 0; z < kernel_size; ++z, in_addr_base0 += input_stride_z, we_addr_base0 += kernel_stride_z)
+                for(uint32_t z = 0; z < kernel_size; ++z, in_addr_base0 += input_stride_z, we_addr_base0 += kernel_stride_z)
                 {
                     const int in_z = id.z() * conv_stride_y + z - conv_pad_top;
 
@@ -586,7 +586,7 @@
                         auto in_addr_base1 = in_addr_base0;
                         auto we_addr_base1 = we_addr_base0;
 
-                        for(int y = 0; y < kernel_size; ++y, in_addr_base1 += input_stride_y, we_addr_base1 += kernel_stride_y)
+                        for(uint32_t y = 0; y < kernel_size; ++y, in_addr_base1 += input_stride_y, we_addr_base1 += kernel_stride_y)
                         {
                             auto out_values = internal_vdupq_n(zero);