bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: new module 'fenv-rounding'


From: Bruno Haible
Subject: Re: new module 'fenv-rounding'
Date: Sat, 04 Nov 2023 14:25:49 +0100

On NetBSD 8.0/sparc64, I see a test failure of the fenv-rounding test for
'long double' operations. On this platform, 'long double' operations are
implemented by a software floating-point library. Apparently this library
code ignores the rounding direction.

This patch avoids the test failure and documents the problem.


2023-11-04  Bruno Haible  <bruno@clisp.org>

        fenv-rounding: Avoid a test failure on NetBSD/sparc64.
        * tests/test-fenv-round.c (test_towardzero, test_upward, test_downward):
        Skip the 'long double' tests on NetBSD/sparc64.
        * doc/posix-functions/fesetround.texi: Mention the problem with
        'long double' operations on some platforms.

diff --git a/doc/posix-functions/fesetround.texi 
b/doc/posix-functions/fesetround.texi
index f0c74c3467..73d6c034fa 100644
--- a/doc/posix-functions/fesetround.texi
+++ b/doc/posix-functions/fesetround.texi
@@ -18,4 +18,7 @@
 
 Portability problems not fixed by Gnulib:
 @itemize
+@item
+The rounding mode has no effect on @code{long double} operations on some 
platforms:
+FreeBSD/arm64, NetBSD/sparc64.
 @end itemize
diff --git a/tests/test-fenv-round.c b/tests/test-fenv-round.c
index aba783cc79..b697fb1301 100644
--- a/tests/test-fenv-round.c
+++ b/tests/test-fenv-round.c
@@ -45,7 +45,7 @@ test_towardzero ()
     a = -1; b = 3; q = a / b; p = b * q;
     ASSERT (p > a);
   }
-#if !(defined __FreeBSD__ && defined __aarch64__)
+#if !((defined __FreeBSD__ && defined __aarch64__) || (defined __NetBSD__ && 
defined __sparcv9 /* sparc64 */))
   {
     long double volatile a, b, q, p;
 
@@ -80,7 +80,7 @@ test_upward ()
     a = -1; b = 3; q = a / b; p = b * q;
     ASSERT (p > a);
   }
-#if !(defined __FreeBSD__ && defined __aarch64__)
+#if !((defined __FreeBSD__ && defined __aarch64__) || (defined __NetBSD__ && 
defined __sparcv9 /* sparc64 */))
   {
     long double volatile a, b, q, p;
 
@@ -115,7 +115,7 @@ test_downward ()
     a = -1; b = 3; q = a / b; p = b * q;
     ASSERT (p < a);
   }
-#if !(defined __FreeBSD__ && defined __aarch64__)
+#if !((defined __FreeBSD__ && defined __aarch64__) || (defined __NetBSD__ && 
defined __sparcv9 /* sparc64 */))
   {
     long double volatile a, b, q, p;
 






reply via email to

[Prev in Thread] Current Thread [Next in Thread]