From: David Hildenbrand <david@redhat.com>
To: Richard Henderson <richard.henderson@linaro.org>, qemu-devel@nongnu.org
Cc: alex.bennee@linaro.org
Subject: Re: [PATCH 50/72] softfloat: Move minmax_flags to softfloat-parts.c.inc
Date: Mon, 17 May 2021 15:14:44 +0200 [thread overview]
Message-ID: <4a35ce8e-941a-b067-89c3-b655061aeb13@redhat.com> (raw)
In-Reply-To: <20210508014802.892561-51-richard.henderson@linaro.org>
On 08.05.21 03:47, 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 types.
>
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
> fpu/softfloat.c | 216 ++++++++++++++++----------------------
> fpu/softfloat-parts.c.inc | 69 ++++++++++++
> 2 files changed, 158 insertions(+), 127 deletions(-)
>
> diff --git a/fpu/softfloat.c b/fpu/softfloat.c
> index 586ea5d67a..4c04e88a3a 100644
> --- a/fpu/softfloat.c
> +++ b/fpu/softfloat.c
> @@ -482,6 +482,15 @@ enum {
> float_cmask_anynan = float_cmask_qnan | float_cmask_snan,
> };
>
> +/* Flags for parts_minmax. */
> +enum {
> + /* Set for minimum; clear for maximum. */
> + minmax_ismin = 1,
> + /* Set for the IEEE 754-2008 minNum() and maxNum() operations. */
> + minmax_isnum = 2,
> + /* Set for the IEEE 754-2008 minNumMag() and minNumMag() operations. */
> + minmax_ismag = 4 | minmax_isnum
> +};
>
> /* Simple helpers for checking if, or what kind of, NaN we have */
> static inline __attribute__((unused)) bool is_nan(FloatClass c)
> @@ -864,6 +873,14 @@ static void parts128_uint_to_float(FloatParts128 *p, uint64_t a,
> #define parts_uint_to_float(P, I, Z, S) \
> PARTS_GENERIC_64_128(uint_to_float, P)(P, I, Z, S)
>
> +static int parts64_minmax(FloatParts64 *a, FloatParts64 *b,
> + float_status *s, int flags, const FloatFmt *fmt);
> +static int parts128_minmax(FloatParts128 *a, FloatParts128 *b,
> + float_status *s, int flags, const FloatFmt *fmt);
> +
> +#define parts_minmax(A, B, S, Z, F) \
> + PARTS_GENERIC_64_128(minmax, A)(A, B, S, Z, F)
> +
> /*
> * Helper functions for softfloat-parts.c.inc, per-size operations.
> */
> @@ -3257,145 +3274,90 @@ float128 uint64_to_float128(uint64_t a, float_status *status)
> return float128_round_pack_canonical(&p, status);
> }
>
> -/* Float Min/Max */
> -/* min() and max() functions. These can't be implemented as
> - * 'compare and pick one input' because that would mishandle
> - * NaNs and +0 vs -0.
> - *
> - * minnum() and maxnum() functions. These are similar to the min()
> - * and max() functions but if one of the arguments is a QNaN and
> - * the other is numerical then the numerical argument is returned.
> - * SNaNs will get quietened before being returned.
> - * minnum() and maxnum correspond to the IEEE 754-2008 minNum()
> - * and maxNum() operations. min() and max() are the typical min/max
> - * semantics provided by many CPUs which predate that specification.
> - *
> - * minnummag() and maxnummag() functions correspond to minNumMag()
> - * and minNumMag() from the IEEE-754 2008.
> +/*
> + * Minimum and maximum
> */
> -static FloatParts64 minmax_floats(FloatParts64 a, FloatParts64 b, bool ismin,
> - bool ieee, bool ismag, float_status *s)
> +
> +static float16 float16_minmax(float16 a, float16 b, float_status *s, int flags)
> {
> - if (unlikely(is_nan(a.cls) || is_nan(b.cls))) {
> - if (ieee) {
> - /* Takes two floating-point values `a' and `b', one of
> - * which is a NaN, and returns the appropriate NaN
> - * result. If either `a' or `b' is a signaling NaN,
> - * the invalid exception is raised.
> - */
> - if (is_snan(a.cls) || is_snan(b.cls)) {
> - return *parts_pick_nan(&a, &b, s);
> - } else if (is_nan(a.cls) && !is_nan(b.cls)) {
> - return b;
> - } else if (is_nan(b.cls) && !is_nan(a.cls)) {
> - return a;
> - }
> - }
> - return *parts_pick_nan(&a, &b, s);
> - } else {
> - int a_exp, b_exp;
> + FloatParts64 pa, pb;
> + int which;
>
> - switch (a.cls) {
> - case float_class_normal:
> - a_exp = a.exp;
> - break;
> - case float_class_inf:
> - a_exp = INT_MAX;
> - break;
> - case float_class_zero:
> - a_exp = INT_MIN;
> - break;
> - default:
> - g_assert_not_reached();
> - break;
> - }
> - switch (b.cls) {
> - case float_class_normal:
> - b_exp = b.exp;
> - break;
> - case float_class_inf:
> - b_exp = INT_MAX;
> - break;
> - case float_class_zero:
> - b_exp = INT_MIN;
> - break;
> - default:
> - g_assert_not_reached();
> - break;
> - }
> -
> - if (ismag && (a_exp != b_exp || a.frac != b.frac)) {
> - bool a_less = a_exp < b_exp;
> - if (a_exp == b_exp) {
> - a_less = a.frac < b.frac;
> - }
> - return a_less ^ ismin ? b : a;
> - }
> -
> - if (a.sign == b.sign) {
> - bool a_less = a_exp < b_exp;
> - if (a_exp == b_exp) {
> - a_less = a.frac < b.frac;
> - }
> - return a.sign ^ a_less ^ ismin ? b : a;
> - } else {
> - return a.sign ^ ismin ? b : a;
> - }
> + float16_unpack_canonical(&pa, a, s);
> + float16_unpack_canonical(&pb, b, s);
> + which = parts_minmax(&pa, &pb, s, flags, &float16_params);
> + if (unlikely(which < 0)) {
> + /* Some sort of nan, need to repack default and silenced nans. */
> + return float16_round_pack_canonical(&pa, s);
> }
> + return which ? b : a;
> }
>
> -#define MINMAX(sz, name, ismin, isiee, ismag) \
> -float ## sz float ## sz ## _ ## name(float ## sz a, float ## sz b, \
> - float_status *s) \
> -{ \
> - FloatParts64 pa, pb, pr; \
> - float ## sz ## _unpack_canonical(&pa, a, s); \
> - float ## sz ## _unpack_canonical(&pb, b, s); \
> - pr = minmax_floats(pa, pb, ismin, isiee, ismag, s); \
> - return float ## sz ## _round_pack_canonical(&pr, s); \
> +static bfloat16 bfloat16_minmax(bfloat16 a, bfloat16 b,
> + float_status *s, int flags)
> +{
> + FloatParts64 pa, pb;
> + int which;
> +
> + bfloat16_unpack_canonical(&pa, a, s);
> + bfloat16_unpack_canonical(&pb, b, s);
> + which = parts_minmax(&pa, &pb, s, flags, &float16_params);
> + if (unlikely(which < 0)) {
> + /* Some sort of nan, need to repack default and silenced nans. */
> + return bfloat16_round_pack_canonical(&pa, s);
> + }
> + return which ? b : a;
> }
>
> -MINMAX(16, min, true, false, false)
> -MINMAX(16, minnum, true, true, false)
> -MINMAX(16, minnummag, true, true, true)
> -MINMAX(16, max, false, false, false)
> -MINMAX(16, maxnum, false, true, false)
> -MINMAX(16, maxnummag, false, true, true)
> +static float32 float32_minmax(float32 a, float32 b, float_status *s, int flags)
> +{
> + FloatParts64 pa, pb;
> + int which;
>
> -MINMAX(32, min, true, false, false)
> -MINMAX(32, minnum, true, true, false)
> -MINMAX(32, minnummag, true, true, true)
> -MINMAX(32, max, false, false, false)
> -MINMAX(32, maxnum, false, true, false)
> -MINMAX(32, maxnummag, false, true, true)
> -
> -MINMAX(64, min, true, false, false)
> -MINMAX(64, minnum, true, true, false)
> -MINMAX(64, minnummag, true, true, true)
> -MINMAX(64, max, false, false, false)
> -MINMAX(64, maxnum, false, true, false)
> -MINMAX(64, maxnummag, false, true, true)
> -
> -#undef MINMAX
> -
> -#define BF16_MINMAX(name, ismin, isiee, ismag) \
> -bfloat16 bfloat16_ ## name(bfloat16 a, bfloat16 b, float_status *s) \
> -{ \
> - FloatParts64 pa, pb, pr; \
> - bfloat16_unpack_canonical(&pa, a, s); \
> - bfloat16_unpack_canonical(&pb, b, s); \
> - pr = minmax_floats(pa, pb, ismin, isiee, ismag, s); \
> - return bfloat16_round_pack_canonical(&pr, s); \
> + float32_unpack_canonical(&pa, a, s);
> + float32_unpack_canonical(&pb, b, s);
> + 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 ? b : a;
> }
>
> -BF16_MINMAX(min, true, false, false)
> -BF16_MINMAX(minnum, true, true, false)
> -BF16_MINMAX(minnummag, true, true, true)
> -BF16_MINMAX(max, false, false, false)
> -BF16_MINMAX(maxnum, false, true, false)
> -BF16_MINMAX(maxnummag, false, true, true)
> +static float64 float64_minmax(float64 a, float64 b, float_status *s, int flags)
> +{
> + FloatParts64 pa, pb;
> + int which;
>
> -#undef BF16_MINMAX
> + float64_unpack_canonical(&pa, a, s);
> + float64_unpack_canonical(&pb, b, s);
> + which = parts_minmax(&pa, &pb, s, flags, &float64_params);
> + if (unlikely(which < 0)) {
> + /* Some sort of nan, need to repack default and silenced nans. */
> + return float64_round_pack_canonical(&pa, s);
> + }
> + return which ? b : a;
> +}
> +
> +#define MINMAX_1(type, name, flags) \
> + type type##_##name(type a, type b, float_status *s) \
> + { return type##_minmax(a, b, s, flags); }
> +
> +#define MINMAX_2(type) \
> + MINMAX_1(type, max, 0) \
> + MINMAX_1(type, maxnum, minmax_isnum) \
> + MINMAX_1(type, maxnummag, minmax_ismag) \
> + MINMAX_1(type, min, minmax_ismin) \
> + MINMAX_1(type, minnum, minmax_ismin | minmax_isnum) \
> + MINMAX_1(type, minnummag, minmax_ismin | minmax_ismag)
> +
> +MINMAX_2(float16)
> +MINMAX_2(bfloat16)
> +MINMAX_2(float32)
> +MINMAX_2(float64)
> +
> +#undef MINMAX_1
> +#undef MINMAX_2
>
> /* Floating point compare */
> static FloatRelation compare_floats(FloatParts64 a, FloatParts64 b, bool is_quiet,
> diff --git a/fpu/softfloat-parts.c.inc b/fpu/softfloat-parts.c.inc
> index f3c4f8c8d2..4d91ef0d32 100644
> --- a/fpu/softfloat-parts.c.inc
> +++ b/fpu/softfloat-parts.c.inc
> @@ -936,3 +936,72 @@ static void partsN(uint_to_float)(FloatPartsN *p, uint64_t a,
> p->frac_hi = a << shift;
> }
> }
> +
> +/*
> + * Float min/max.
> + *
> + * Return -1 to return the chosen nan in *a;
> + * return 0 to use the a input unchanged; 1 to use the b input unchanged.
> + */
> +static int partsN(minmax)(FloatPartsN *a, FloatPartsN *b,
> + float_status *s, int flags, const FloatFmt *fmt)
> +{
> + int ab_mask = float_cmask(a->cls) | float_cmask(b->cls);
> + int a_exp, b_exp;
> + bool a_less;
> +
> + if (unlikely(ab_mask & float_cmask_anynan)) {
> + /*
> + * For minnum/maxnum, if one operand is a QNaN, and the other
> + * operand is numerical, then return numerical argument.
> + */
> + if ((flags & minmax_isnum)
> + && !(ab_mask & float_cmask_snan)
> + && (ab_mask & ~float_cmask_qnan)) {
> + return is_nan(a->cls);
> + }
> + *a = *parts_pick_nan(a, b, s);
> + return -1;
> + }
> +
> + a_exp = a->exp;
> + b_exp = b->exp;
> +
> + if (unlikely(ab_mask != float_cmask_normal)) {
> + switch (a->cls) {
> + case float_class_normal:
> + break;
> + case float_class_inf:
> + a_exp = INT_MAX;
> + break;
> + case float_class_zero:
> + a_exp = INT_MIN;
> + break;
> + default:
> + g_assert_not_reached();
> + break;
> + }
> + switch (b->cls) {
> + case float_class_normal:
> + break;
> + case float_class_inf:
> + b_exp = INT_MAX;
> + break;
> + case float_class_zero:
> + b_exp = INT_MIN;
> + break;
> + default:
> + g_assert_not_reached();
> + break;
> + }
> + }
> +
> + if (a->sign != b->sign && !(flags & minmax_ismag)) {
> + a_less = a->sign;
> + } else if (a_exp != b_exp) {
> + a_less = a_exp < b_exp;
> + } else {
> + a_less = frac_cmp(a, b) < 0;
> + }
> + return a_less ^ !!(flags & minmax_ismin);
> +}
This patch introduces two issues:
1. Comparing two negative numbers is broken. We have to
invert the a_less result.
2. The check "flags & minmax_ismag" is broken because
"minmax_ismag = 4 | minmax_isnum" and it, therefore,
also triggers for "flags = minmax_isnum"
The following seems to make it work in my tests:
diff --git a/fpu/softfloat-parts.c.inc b/fpu/softfloat-parts.c.inc
index 1bc8db4cf1..d5a73dac00 100644
--- a/fpu/softfloat-parts.c.inc
+++ b/fpu/softfloat-parts.c.inc
@@ -1273,6 +1273,9 @@ static int partsN(minmax)(FloatPartsN *a, FloatPartsN *b,
} else {
a_less = frac_cmp(a, b) < 0;
}
+ if (a->sign && b->sign && !(flags & minmax_ismag)) {
+ a_less = !a_less;
+ }
return a_less ^ !!(flags & minmax_ismin);
}
diff --git a/fpu/softfloat.c b/fpu/softfloat.c
index bfe5a6b975..649e8c6592 100644
--- a/fpu/softfloat.c
+++ b/fpu/softfloat.c
@@ -435,7 +435,7 @@ enum {
/* Set for the IEEE 754-2008 minNum() and maxNum() operations. */
minmax_isnum = 2,
/* Set for the IEEE 754-2008 minNumMag() and minNumMag() operations. */
- minmax_ismag = 4 | minmax_isnum
+ minmax_ismag = 4,
};
/* Simple helpers for checking if, or what kind of, NaN we have */
@@ -3916,10 +3916,10 @@ static float128 float128_minmax(float128 a, float128 b, float_status *s,
#define MINMAX_2(type) \
MINMAX_1(type, max, 0) \
MINMAX_1(type, maxnum, minmax_isnum) \
- MINMAX_1(type, maxnummag, minmax_ismag) \
+ MINMAX_1(type, maxnummag, minmax_isnum | minmax_ismag) \
MINMAX_1(type, min, minmax_ismin) \
MINMAX_1(type, minnum, minmax_ismin | minmax_isnum) \
- MINMAX_1(type, minnummag, minmax_ismin | minmax_ismag)
+ MINMAX_1(type, minnummag, minmax_ismin | minmax_isnum | minmax_ismag)
MINMAX_2(float16)
MINMAX_2(bfloat16)
--
Thanks,
David / dhildenb
next prev parent reply other threads:[~2021-05-17 13:17 UTC|newest]
Thread overview: 151+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-08 1:46 [PATCH 00/72] Convert floatx80 and float128 to FloatParts Richard Henderson
2021-05-08 1:46 ` [PATCH 01/72] qemu/host-utils: Use __builtin_bitreverseN Richard Henderson
2021-05-10 9:59 ` Alex Bennée
2021-05-11 9:41 ` David Hildenbrand
2021-05-08 1:46 ` [PATCH 02/72] qemu/host-utils: Add wrappers for overflow builtins Richard Henderson
2021-05-10 10:22 ` Alex Bennée
2021-05-08 1:46 ` [PATCH 03/72] qemu/host-utils: Add wrappers for carry builtins Richard Henderson
2021-05-10 12:57 ` Alex Bennée
2021-05-11 20:10 ` Richard Henderson
2021-05-12 11:17 ` Alex Bennée
2021-05-08 1:46 ` [PATCH 04/72] accel/tcg: Use add/sub overflow routines in tcg-runtime-gvec.c Richard Henderson
2021-05-09 8:43 ` Philippe Mathieu-Daudé
2021-05-10 13:02 ` Alex Bennée
2021-05-11 9:46 ` David Hildenbrand
2021-05-08 1:46 ` [PATCH 05/72] tests/fp: add quad support to the benchmark utility Richard Henderson
2021-05-11 10:01 ` David Hildenbrand
2021-05-08 1:46 ` [PATCH 06/72] softfloat: Move the binary point to the msb Richard Henderson
2021-05-10 13:36 ` Alex Bennée
2021-05-08 1:46 ` [PATCH 07/72] softfloat: Inline float_raise Richard Henderson
2021-05-09 8:32 ` Philippe Mathieu-Daudé
2021-05-11 10:04 ` David Hildenbrand
2021-05-08 1:46 ` [PATCH 08/72] softfloat: Use float_raise in more places Richard Henderson
2021-05-09 8:34 ` Philippe Mathieu-Daudé
2021-05-11 10:06 ` David Hildenbrand
2021-05-08 1:46 ` [PATCH 09/72] softfloat: Tidy a * b + inf return Richard Henderson
2021-05-08 1:47 ` [PATCH 10/72] softfloat: Add float_cmask and constants Richard Henderson
2021-05-08 1:47 ` [PATCH 11/72] softfloat: Use return_nan in float_to_float Richard Henderson
2021-05-10 15:10 ` Alex Bennée
2021-05-11 10:10 ` David Hildenbrand
2021-05-08 1:47 ` [PATCH 12/72] softfloat: fix return_nan vs default_nan_mode Richard Henderson
2021-05-10 15:12 ` Alex Bennée
2021-05-11 10:12 ` David Hildenbrand
2021-05-08 1:47 ` [PATCH 13/72] target/mips: Set set_default_nan_mode with set_snan_bit_is_one Richard Henderson
2021-05-11 9:37 ` Alex Bennée
2021-05-11 10:16 ` David Hildenbrand
2021-05-08 1:47 ` [PATCH 14/72] softfloat: Do not produce a default_nan from parts_silence_nan Richard Henderson
2021-05-11 10:16 ` David Hildenbrand
2021-05-11 10:32 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 15/72] softfloat: Rename FloatParts to FloatParts64 Richard Henderson
2021-05-09 8:45 ` Philippe Mathieu-Daudé
2021-05-11 10:16 ` David Hildenbrand
2021-05-08 1:47 ` [PATCH 16/72] softfloat: Move type-specific pack/unpack routines Richard Henderson
2021-05-09 8:46 ` Philippe Mathieu-Daudé
2021-05-11 10:17 ` David Hildenbrand
2021-05-08 1:47 ` [PATCH 17/72] softfloat: Use pointers with parts_default_nan Richard Henderson
2021-05-11 10:22 ` David Hildenbrand
2021-05-11 20:19 ` Richard Henderson
2021-05-08 1:47 ` [PATCH 18/72] softfloat: Use pointers with unpack_raw Richard Henderson
2021-05-09 8:48 ` Philippe Mathieu-Daudé
2021-05-12 12:58 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 19/72] softfloat: Use pointers with ftype_unpack_raw Richard Henderson
2021-05-11 11:31 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 20/72] softfloat: Use pointers with pack_raw Richard Henderson
2021-05-11 11:32 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 21/72] softfloat: Use pointers with ftype_pack_raw Richard Henderson
2021-05-11 11:32 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 22/72] softfloat: Use pointers with ftype_unpack_canonical Richard Henderson
2021-05-11 13:54 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 23/72] softfloat: Use pointers with ftype_round_pack_canonical Richard Henderson
2021-05-11 13:55 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 24/72] softfloat: Use pointers with parts_silence_nan Richard Henderson
2021-05-11 13:56 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 25/72] softfloat: Rearrange FloatParts64 Richard Henderson
2021-05-11 13:57 ` Alex Bennée
2021-05-11 15:04 ` Richard Henderson
2021-05-12 11:08 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 26/72] softfloat: Convert float128_silence_nan to parts Richard Henderson
2021-05-13 8:34 ` Alex Bennée
2021-05-13 12:25 ` Richard Henderson
2021-05-08 1:47 ` [PATCH 27/72] softfloat: Convert float128_default_nan " Richard Henderson
2021-05-13 8:56 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 28/72] softfloat: Move return_nan to softfloat-parts.c.inc Richard Henderson
2021-05-12 18:10 ` David Hildenbrand
2021-05-08 1:47 ` [PATCH 29/72] softfloat: Move pick_nan " Richard Henderson
2021-05-12 18:16 ` David Hildenbrand
2021-05-13 12:28 ` Richard Henderson
2021-05-08 1:47 ` [PATCH 30/72] softfloat: Move pick_nan_muladd " Richard Henderson
2021-05-12 18:18 ` David Hildenbrand
2021-05-08 1:47 ` [PATCH 31/72] softfloat: Move sf_canonicalize " Richard Henderson
2021-05-13 9:45 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 32/72] softfloat: Move round_canonical " Richard Henderson
2021-05-13 9:53 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 33/72] softfloat: Use uadd64_carry, usub64_borrow in softfloat-macros.h Richard Henderson
2021-05-13 10:00 ` Alex Bennée
2021-05-13 12:38 ` Richard Henderson
2021-05-08 1:47 ` [PATCH 34/72] softfloat: Move addsub_floats to softfloat-parts.c.inc Richard Henderson
2021-05-13 10:03 ` Alex Bennée
2021-05-13 10:05 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 35/72] softfloat: Implement float128_add/sub via parts Richard Henderson
2021-05-13 10:06 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 36/72] softfloat: Move mul_floats to softfloat-parts.c.inc Richard Henderson
2021-05-13 10:08 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 37/72] softfloat: Move muladd_floats " Richard Henderson
2021-05-13 10:43 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 38/72] softfloat: Use mulu64 for mul64To128 Richard Henderson
2021-05-08 1:47 ` [PATCH 39/72] softfloat: Use add192 in mul128To256 Richard Henderson
2021-05-13 10:49 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 40/72] softfloat: Tidy mul128By64To192 Richard Henderson
2021-05-13 10:50 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 41/72] softfloat: Introduce sh[lr]_double primitives Richard Henderson
2021-05-13 10:59 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 42/72] softfloat: Move div_floats to softfloat-parts.c.inc Richard Henderson
2021-05-13 11:02 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 43/72] softfloat: Split float_to_float Richard Henderson
2021-05-13 11:04 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 44/72] softfloat: Convert float-to-float conversions with float128 Richard Henderson
2021-05-13 11:17 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 45/72] softfloat: Move round_to_int to softfloat-parts.c.inc Richard Henderson
2021-05-13 14:10 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 46/72] softfloat: Move rount_to_int_and_pack " Richard Henderson
2021-05-13 14:11 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 47/72] softfloat: Move rount_to_uint_and_pack " Richard Henderson
2021-05-08 1:47 ` [PATCH 48/72] softfloat: Move int_to_float " Richard Henderson
2021-05-08 1:47 ` [PATCH 49/72] softfloat: Move uint_to_float " Richard Henderson
2021-05-08 1:47 ` [PATCH 50/72] softfloat: Move minmax_flags " Richard Henderson
2021-05-17 13:14 ` David Hildenbrand [this message]
2021-05-17 15:57 ` Richard Henderson
2021-05-08 1:47 ` [PATCH 51/72] softfloat: Move compare_floats " Richard Henderson
2021-05-08 1:47 ` [PATCH 52/72] softfloat: Move scalbn_decomposed " Richard Henderson
2021-05-08 1:47 ` [PATCH 53/72] softfloat: Move sqrt_float " Richard Henderson
2021-05-08 1:47 ` [PATCH 54/72] softfloat: Split out parts_uncanon_normal Richard Henderson
2021-05-08 1:47 ` [PATCH 55/72] softfloat: Reduce FloatFmt Richard Henderson
2021-05-08 1:47 ` [PATCH 56/72] softfloat: Introduce Floatx80RoundPrec Richard Henderson
2021-05-08 1:47 ` [PATCH 57/72] softfloat: Adjust parts_uncanon_normal for floatx80 Richard Henderson
2021-05-08 1:47 ` [PATCH 58/72] tests/fp/fp-test: Reverse order of floatx80 precision tests Richard Henderson
2021-05-13 14:12 ` Alex Bennée
2021-05-08 1:47 ` [PATCH 59/72] softfloat: Convert floatx80_add/sub to FloatParts Richard Henderson
2021-05-08 1:47 ` [PATCH 60/72] softfloat: Convert floatx80_mul " Richard Henderson
2021-05-08 1:47 ` [PATCH 61/72] softfloat: Convert floatx80_div " Richard Henderson
2021-05-08 1:47 ` [PATCH 62/72] softfloat: Convert floatx80_sqrt " Richard Henderson
2021-05-08 1:47 ` [PATCH 63/72] softfloat: Convert floatx80_round " Richard Henderson
2021-05-08 1:47 ` [PATCH 64/72] softfloat: Convert floatx80_round_to_int " Richard Henderson
2021-05-08 1:47 ` [PATCH 65/72] softfloat: Convert integer to floatx80 " Richard Henderson
2021-05-08 1:47 ` [PATCH 66/72] softfloat: Convert floatx80 float conversions " Richard Henderson
2021-05-08 1:47 ` [PATCH 67/72] softfloat: Convert floatx80 to integer " Richard Henderson
2021-05-08 1:47 ` [PATCH 68/72] softfloat: Convert floatx80_scalbn " Richard Henderson
2021-05-08 1:47 ` [PATCH 69/72] softfloat: Convert floatx80 compare " Richard Henderson
2021-05-08 1:48 ` [PATCH 70/72] softfloat: Convert float32_exp2 " Richard Henderson
2021-05-08 1:48 ` [PATCH 71/72] softfloat: Move floatN_log2 to softfloat-parts.c.inc Richard Henderson
2021-05-08 1:48 ` [PATCH 72/72] softfloat: Convert modrem operations to FloatParts Richard Henderson
2021-05-08 2:52 ` [PATCH 00/72] Convert floatx80 and float128 " no-reply
2021-05-10 13:36 ` Alex Bennée
2021-05-12 1:52 ` Richard Henderson
2021-05-12 11:22 ` Alex Bennée
2021-05-12 15:28 ` Richard Henderson
2021-05-12 16:47 ` Alex Bennée
2021-05-12 19:23 ` Alex Bennée
2021-05-13 11:49 ` Richard Henderson
2021-05-13 13:33 ` Alex Bennée
2021-05-13 23:54 ` Richard Henderson
2021-05-13 14:13 ` Alex Bennée
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4a35ce8e-941a-b067-89c3-b655061aeb13@redhat.com \
--to=david@redhat.com \
--cc=alex.bennee@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=richard.henderson@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).