]> git.ipfire.org Git - thirdparty/glibc.git/commitdiff
math: Fix compare sort function on compoundn
authorAdhemerval Zanella <adhemerval.zanella@linaro.org>
Mon, 20 Oct 2025 12:27:53 +0000 (09:27 -0300)
committerAdhemerval Zanella <adhemerval.zanella@linaro.org>
Tue, 21 Oct 2025 12:27:05 +0000 (09:27 -0300)
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 <collin.funk1@gmail.com>
math/s_compoundn_template.c

index ddbcc145809a5abf8d617a1fa64a1c2965a3b446..224610ba837a789980e0417b27e1a347c4cd1e1a 100644 (file)
@@ -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)