Re: [PATCH v2 04/28] softfloat: Move minmax_flags to softfloat-parts.c.inc

2021-06-02 Thread Richard Henderson
On 5/25/21 8:06 AM, Richard Henderson wrote: +which = parts_minmax(&pa, &pb, s, flags, &float32_params); +if (unlikely(which < 0)) { +/* Some sort of nan, need to repack default and silenced nans. */ +return float32_round_pack_canonical(&pa, s); +} +return which ?

Re: [PATCH v2 04/28] softfloat: Move minmax_flags to softfloat-parts.c.inc

2021-06-02 Thread Richard Henderson
On 6/2/21 1:36 PM, Alex Bennée wrote: Richard Henderson writes: Rename to parts$N_minmax. Combine 3 bool arguments to a bitmask, return a tri-state value to indicate nan vs unchanged operand. Introduce ftype_minmax functions as a common optimization point. Fold bfloat16 expansions into the s

Re: [PATCH v2 04/28] softfloat: Move minmax_flags to softfloat-parts.c.inc

2021-06-02 Thread Alex Bennée
Richard Henderson writes: > Rename to parts$N_minmax. Combine 3 bool arguments to a bitmask, > return a tri-state value to indicate nan vs unchanged operand. > Introduce ftype_minmax functions as a common optimization point. > Fold bfloat16 expansions into the same macro as the other types. >

Re: [PATCH v2 04/28] softfloat: Move minmax_flags to softfloat-parts.c.inc

2021-05-26 Thread David Hildenbrand
On 25.05.21 17:06, Richard Henderson wrote: Rename to parts$N_minmax. Combine 3 bool arguments to a bitmask, return a tri-state value to indicate nan vs unchanged operand. Introduce ftype_minmax functions as a common optimization point. Fold bfloat16 expansions into the same macro as the other t

[PATCH v2 04/28] softfloat: Move minmax_flags to softfloat-parts.c.inc

2021-05-25 Thread Richard Henderson
Rename to parts$N_minmax. Combine 3 bool arguments to a bitmask, return a tri-state value to indicate nan vs unchanged operand. Introduce ftype_minmax functions as a common optimization point. Fold bfloat16 expansions into the same macro as the other types. Signed-off-by: Richard Henderson ---