Int32 to Float32 implicit casting fix

- Int32 is cast to Float32 implicitly during range checking comparisons, value may change due to float32 range limitations.
- Fixed by casting float32 var to double64 before comparisons.
- This issue prevented NDK r23b compilation

Resolves: [COMPMID-5051]
Change-Id: I1695f9ad5fa147d81cbcfda9e0152aefab960c82
Signed-off-by: Mohammed Suhail Munshi <MohammedSuhail.Munshi@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/7093
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Sheri Zhang <sheri.zhang@arm.com>

1 files changed, 4 insertions, 2 deletions

diff --git a/arm_compute/core/Utils.h b/arm_compute/core/Utils.h
index b24955d778..a279ef310a 100644
--- a/arm_compute/core/Utils.h
+++ b/arm_compute/core/Utils.h
@@ -1156,13 +1156,15 @@ bool check_value_range(T val, DataType dt, QuantizationInfo qinfo = Quantization
         }
         case DataType::U32:
         {
+            const auto val_d64 = static_cast<double>(val);
             const auto val_u32 = static_cast<uint32_t>(val);
-            return ((val_u32 == val) && val >= std::numeric_limits<uint32_t>::lowest() && val <= std::numeric_limits<uint32_t>::max());
+            return ((val_u32 == val_d64) && val_d64 >= std::numeric_limits<uint32_t>::lowest() && val_d64 <= std::numeric_limits<uint32_t>::max());
         }
         case DataType::S32:
         {
+            const auto val_d64 = static_cast<double>(val);
             const auto val_s32 = static_cast<int32_t>(val);
-            return ((val_s32 == val) && val >= std::numeric_limits<int32_t>::lowest() && val <= std::numeric_limits<int32_t>::max());
+            return ((val_s32 == val_d64) && val_d64 >= std::numeric_limits<int32_t>::lowest() && val_d64 <= std::numeric_limits<int32_t>::max());
         }
         case DataType::BFLOAT16:
             return (val >= bfloat16::lowest() && val <= bfloat16::max());