commit 58e9e06102da7042bed34482ae89b3a6f8c77dca
author Pablo Marquez Tello <pablo.tello@arm.com> Tue Oct 05 16:48:57 2021 +0100
committer Pablo Marquez Tello <pablo.tello@arm.com> Wed Oct 06 15:53:31 2021 +0000
tree 4023e59d4f4b7f6f9da620e0e5322a45ffbb05d1
parent 1024d9f8b4d94266d7a8d7376232a68c251e3f38

Fix the multipliers calculation

* Removed code forcing the multipliers and shifts to 0
* Resolves COMPMID-4833

Change-Id: I7c0eadde3691732c263818bb1cd0fc0cb75ce5a9
Signed-off-by: Pablo Marquez Tello <pablo.tello@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/6378
Reviewed-by: Giorgio Arena <giorgio.arena@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>

src/core/utils/quantization/AsymmHelpers.cpp

diff --git a/src/core/utils/quantization/AsymmHelpers.cpp b/src/core/utils/quantization/AsymmHelpers.cpp
index 81e813c..4ce6099 100644
--- a/src/core/utils/quantization/AsymmHelpers.cpp
+++ b/src/core/utils/quantization/AsymmHelpers.cpp
@@ -61,12 +61,6 @@
     ARM_COMPUTE_RETURN_ERROR_ON(right_shift == nullptr);
     ARM_COMPUTE_RETURN_ERROR_ON(multiplier < -internal_epsilon);
     ARM_COMPUTE_RETURN_ERROR_ON(multiplier > 1.0f + internal_epsilon);
-    if(std::fabs(0.0f - multiplier) < internal_epsilon)
-    {
-        *quant_multiplier = 0;
-        *right_shift      = 0;
-        return Status{};
-    }
 
     int          shift_exp = 0;
     const double q         = std::frexp(multiplier, &shift_exp);