On 12/8/2024 4:48 PM, Richard Henderson wrote:
Initialize x with accumulated via direct assignment,
rather than multiplying by 1.

Signed-off-by: Richard Henderson <richard.hender...@linaro.org>
---
  target/hexagon/fma_emu.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/target/hexagon/fma_emu.c b/target/hexagon/fma_emu.c
index 6749538c09..bc6e742639 100644
--- a/target/hexagon/fma_emu.c
+++ b/target/hexagon/fma_emu.c
@@ -453,7 +453,7 @@ float64 internal_mpyhh(float64 a, float64 b,
          float64_is_infinity(b)) {
          return float64_mul(a, b, fp_status);
      }
-    x.mant = int128_mul_6464(accumulated, 1);
+    x.mant = int128_make64(accumulated);
      x.sticky = sticky;
      prod = fGETUWORD(1, float64_getmant(a)) * fGETUWORD(1, 
float64_getmant(b));
      x.mant = int128_add(x.mant, int128_mul_6464(prod, 0x100000000ULL));

Reviewed-by: Brian Cain <brian.c...@oss.qualcomm.com>


Reply via email to