aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRichard Henderson <richard.henderson@linaro.org>2024-12-08 13:51:04 -0600
committerRichard Henderson <richard.henderson@linaro.org>2024-12-24 08:32:15 -0800
commit2eca1928f9afb7cbc8e72a59dffb964c8319469a (patch)
tree0c8e0e45061f23dd3ca12e81a25accbcbda8f228
parent655a83cac128574c7ea386042f8eefa5be5708e5 (diff)
downloadqemu-2eca1928f9afb7cbc8e72a59dffb964c8319469a.zip
qemu-2eca1928f9afb7cbc8e72a59dffb964c8319469a.tar.gz
qemu-2eca1928f9afb7cbc8e72a59dffb964c8319469a.tar.bz2
target/hexagon: Use float32_muladd for helper_sffms
There are no special cases for this instruction. Since hexagon always uses default-nan mode, explicitly negating the first input is unnecessary. Use float_muladd_negate_product instead. Reviewed-by: Brian Cain <brian.cain@oss.qualcomm.com> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
-rw-r--r--target/hexagon/op_helper.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/target/hexagon/op_helper.c b/target/hexagon/op_helper.c
index 15b143a..95bfa5d 100644
--- a/target/hexagon/op_helper.c
+++ b/target/hexagon/op_helper.c
@@ -1096,10 +1096,9 @@ float32 HELPER(sffma_sc)(CPUHexagonState *env, float32 RxV,
float32 HELPER(sffms)(CPUHexagonState *env, float32 RxV,
float32 RsV, float32 RtV)
{
- float32 neg_RsV;
arch_fpop_start(env);
- neg_RsV = float32_set_sign(RsV, float32_is_neg(RsV) ? 0 : 1);
- RxV = internal_fmafx(neg_RsV, RtV, RxV, 0, &env->fp_status);
+ RxV = float32_muladd(RsV, RtV, RxV, float_muladd_negate_product,
+ &env->fp_status);
arch_fpop_end(env);
return RxV;
}