mbox series

[for-10.0,00/11] fpu: pickNaN follow ups

Message ID 20241203203949.483774-1-richard.henderson@linaro.org
Headers show
Series fpu: pickNaN follow ups | expand

Message

Richard Henderson Dec. 3, 2024, 8:39 p.m. UTC
Based-on: 20241202131347.498124-55-peter.maydell@linaro.org
([PATCH v2 for-10.0 54/54] fpu: Remove default handling for dnan_pattern)

The first patch needs to be inserted into Peter's patch set,
any place before 54/54 when dnan_pattern becomes mandatory.

The rest are cleanups that become possible when the
softfloat-specialize.c.inc functions are inlined into their
single callers in softfloat-parts.c.inc.


r~


Richard Henderson (11):
  target/arm: Copy entire float_status in is_ebf
  softfloat: Inline pickNaNMulAdd
  softfloat: Use goto for default nan case in pick_nan_muladd
  softfloat: Remove which from parts_pick_nan_muladd
  softfloat: Pad array size in pick_nan_muladd
  softfloat: Move propagateFloatx80NaN to softfloat.c
  softfloat: Use parts_pick_nan in propagateFloatx80NaN
  softfloat: Inline pickNaN
  softfloat: Share code between parts_pick_nan cases
  softfloat: Sink frac_cmp in parts_pick_nan until needed
  softfloat: Replace WHICH with RET in parts_pick_nan

 fpu/softfloat.c                |  19 ++++
 target/arm/tcg/vec_helper.c    |  20 ++--
 fpu/softfloat-parts.c.inc      | 136 +++++++++++++++++-----
 fpu/softfloat-specialize.c.inc | 202 ---------------------------------
 4 files changed, 131 insertions(+), 246 deletions(-)

Comments

Peter Maydell Dec. 10, 2024, 5:23 p.m. UTC | #1
On Tue, 3 Dec 2024 at 20:39, Richard Henderson
<richard.henderson@linaro.org> wrote:
>
> Based-on: 20241202131347.498124-55-peter.maydell@linaro.org
> ([PATCH v2 for-10.0 54/54] fpu: Remove default handling for dnan_pattern)
>
> The first patch needs to be inserted into Peter's patch set,
> any place before 54/54 when dnan_pattern becomes mandatory.
>
> The rest are cleanups that become possible when the
> softfloat-specialize.c.inc functions are inlined into their
> single callers in softfloat-parts.c.inc.
>

I've queued this to target-arm.next for 10.0.

thanks
-- PMM