From 0e4ca88bd24c1d705a45b5e12fe562aaa9516ef4 Mon Sep 17 00:00:00 2001 From: Adhemerval Zanella Date: Mon, 20 Oct 2025 09:27:53 -0300 Subject: [PATCH] math: Fix compare sort function on compoundn To use the fabs function to the used type, instead of the double variant. it fixes a build issue with clang: ./s_compoundn_template.c:64:14: error: absolute value function 'fabs' given an argument of type 'const long double' but has parameter of type 'double' which may cause truncation of value [-Werror,-Wabsolute-value] 64 | FLOAT pd = fabs (*(const FLOAT *) p); | ^ ./s_compoundn_template.c:64:14: note: use function 'fabsl' instead 64 | FLOAT pd = fabs (*(const FLOAT *) p); | ^~~~ | fabsl Reviewed-by: Collin Funk --- math/s_compoundn_template.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/math/s_compoundn_template.c b/math/s_compoundn_template.c index ddbcc14580..224610ba83 100644 --- a/math/s_compoundn_template.c +++ b/math/s_compoundn_template.c @@ -61,8 +61,8 @@ mul3_split (FLOAT *out, FLOAT a, FLOAT b, FLOAT c, FLOAT d) static int compare (const void *p, const void *q) { - FLOAT pd = fabs (*(const FLOAT *) p); - FLOAT qd = fabs (*(const FLOAT *) q); + FLOAT pd = M_FABS (*(const FLOAT *) p); + FLOAT qd = M_FABS (*(const FLOAT *) q); if (pd < qd) return -1; else if (pd == qd)