From 41d886cecd6478725e9d0da93af7de5847efca1e Mon Sep 17 00:00:00 2001 From: Pablo Marquez Tello Date: Thu, 30 Sep 2021 13:56:17 +0100 Subject: Fixed the compiler warning -Werror=type-limits * Comparison is always false due to limited range of data type. rescale_value is truncated to int32_t and then is compared agains (1ll <<31) which will be always false * Resolves MLCE-508 Change-Id: I252f64d4aec8f3e120d71fc5e40c7665816173ac Signed-off-by: Pablo Marquez Tello Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/6342 Tested-by: Arm Jenkins Reviewed-by: Giorgio Arena Reviewed-by: Gunes Bayir Comments-Addressed: Arm Jenkins --- .../kernels/a64_s8q_nhwc_avg_generic_depthfirst/generic.cpp | 7 ++++--- .../pooling/kernels/a64_u8_nhwc_avg_generic_depthfirst/generic.cpp | 7 ++++--- .../kernels/a64_u8q_nhwc_avg_generic_depthfirst/generic.cpp | 7 ++++--- 3 files changed, 12 insertions(+), 9 deletions(-) (limited to 'src/core/NEON/kernels/arm_conv/pooling') diff --git a/src/core/NEON/kernels/arm_conv/pooling/kernels/a64_s8q_nhwc_avg_generic_depthfirst/generic.cpp b/src/core/NEON/kernels/arm_conv/pooling/kernels/a64_s8q_nhwc_avg_generic_depthfirst/generic.cpp index f288a4119c..530ee06080 100644 --- a/src/core/NEON/kernels/arm_conv/pooling/kernels/a64_s8q_nhwc_avg_generic_depthfirst/generic.cpp +++ b/src/core/NEON/kernels/arm_conv/pooling/kernels/a64_s8q_nhwc_avg_generic_depthfirst/generic.cpp @@ -86,12 +86,13 @@ void a64_s8q_nhwc_avg_generic_depthfirst_impl( f_rescale_value *= 2.0f; } - rescale_value = static_cast(round(f_rescale_value * static_cast(1ll << 31))); - if (static_cast(rescale_value) == (1ll << 31)) + int64_t large_rescale_value = round(f_rescale_value * static_cast(1ll << 31)); + if (large_rescale_value == (1ll << 31)) { shift_value++; - rescale_value >>= 1; + large_rescale_value >>= 1; } + rescale_value = static_cast(large_rescale_value); } // Combine together the rescale value for the requantization and the scaling diff --git a/src/core/NEON/kernels/arm_conv/pooling/kernels/a64_u8_nhwc_avg_generic_depthfirst/generic.cpp b/src/core/NEON/kernels/arm_conv/pooling/kernels/a64_u8_nhwc_avg_generic_depthfirst/generic.cpp index 2c8a29248d..baf23b4a4d 100644 --- a/src/core/NEON/kernels/arm_conv/pooling/kernels/a64_u8_nhwc_avg_generic_depthfirst/generic.cpp +++ b/src/core/NEON/kernels/arm_conv/pooling/kernels/a64_u8_nhwc_avg_generic_depthfirst/generic.cpp @@ -84,12 +84,13 @@ void a64_u8_nhwc_avg_generic_depthfirst_impl( f_rescale_value *= 2.0f; } - rescale_value = static_cast(round(f_rescale_value * static_cast(1ll << 31))); - if (static_cast(rescale_value) == (1ll << 31)) + int64_t large_rescale_value = round(f_rescale_value * static_cast(1ll << 31)); + if (large_rescale_value == (1ll << 31)) { shift_value++; - rescale_value >>= 1; + large_rescale_value >>= 1; } + rescale_value = static_cast(large_rescale_value); } __asm__ __volatile__( diff --git a/src/core/NEON/kernels/arm_conv/pooling/kernels/a64_u8q_nhwc_avg_generic_depthfirst/generic.cpp b/src/core/NEON/kernels/arm_conv/pooling/kernels/a64_u8q_nhwc_avg_generic_depthfirst/generic.cpp index a57fe6df68..11376e0fe2 100644 --- a/src/core/NEON/kernels/arm_conv/pooling/kernels/a64_u8q_nhwc_avg_generic_depthfirst/generic.cpp +++ b/src/core/NEON/kernels/arm_conv/pooling/kernels/a64_u8q_nhwc_avg_generic_depthfirst/generic.cpp @@ -87,12 +87,13 @@ void a64_u8q_nhwc_avg_generic_depthfirst_impl( f_rescale_value *= 2.0f; } - rescale_value = static_cast(round(f_rescale_value * static_cast(1ll << 31))); - if (static_cast(rescale_value) == (1ll << 31)) + int64_t large_rescale_value = round(f_rescale_value * static_cast(1ll << 31)); + if (large_rescale_value == (1ll << 31)) { shift_value++; - rescale_value >>= 1; + large_rescale_value >>= 1; } + rescale_value = static_cast(large_rescale_value); } -- cgit v1.2.1