Re: [PATCH 10/18] fpu: Use get_float_infzeronan_rule in partsN_pick_nan_muladd

2026-05-08 Thread Philippe Mathieu-Daudé 

On 8/5/26 00:17, Richard Henderson wrote:

Deduplicate the uses to a local variable.

Signed-off-by: Richard Henderson 
---
  fpu/softfloat-parts.c.inc | 6 +++---
  1 file changed, 3 insertions(+), 3 deletions(-)


Reviewed-by: Philippe Mathieu-Daudé

[PATCH 10/18] fpu: Use get_float_infzeronan_rule in partsN_pick_nan_muladd

2026-05-07 Thread Richard Henderson 
Deduplicate the uses to a local variable.

Signed-off-by: Richard Henderson 
---
 fpu/softfloat-parts.c.inc | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/fpu/softfloat-parts.c.inc b/fpu/softfloat-parts.c.inc
index 1d8a98358c..38b317c1a7 100644
--- a/fpu/softfloat-parts.c.inc
+++ b/fpu/softfloat-parts.c.inc
@@ -122,14 +122,14 @@ static FloatPartsN partsN(pick_nan_muladd)(const 
FloatPartsN *a,
 {
 bool infzero = (ab_mask == float_cmask_infzero);
 bool have_snan = (abc_mask & float_cmask_snan);
+FloatInfZeroNaNRule izn_rule = get_float_infzeronan_rule(s);
 const FloatPartsN *ret;
 
 if (unlikely(have_snan)) {
 float_raise(float_flag_invalid | float_flag_invalid_snan, s);
 }
 
-if (infzero &&
-!(s->float_infzeronan_rule & float_infzeronan_suppress_invalid)) {
+if (infzero && !(izn_rule & float_infzeronan_suppress_invalid)) {
 /* This is (0 * inf) + NaN or (inf * 0) + NaN */
 float_raise(float_flag_invalid | float_flag_invalid_imz, s);
 }
@@ -147,7 +147,7 @@ static FloatPartsN partsN(pick_nan_muladd)(const 
FloatPartsN *a,
  * Inf * 0 + NaN -- some implementations return the
  * default NaN here, and some return the input NaN.
  */
-switch (s->float_infzeronan_rule & ~float_infzeronan_suppress_invalid) 
{
+switch (izn_rule & ~float_infzeronan_suppress_invalid) {
 case float_infzeronan_dnan_never:
 break;
 case float_infzeronan_dnan_always:
-- 
2.43.0