Re: [PATCH] Fix ceill() for ldbl-96

[Andreas Jaeger <aj at suse dot de>]

Ulrich Drepper <drepper@redhat.com> writes:

> Gwenore Beauchesne wrote:
>> ceil(x) is defined so that ceil(x) = n / n - 1 < x <= n, where n is
>> integral, x a real number. The following patch fixes this for ldbl-96 
>> targets (e.g. AMD64), in particular ceill(0.25).
>
> Test case?

I'll append one.

But that test passes even without the patch for me on amd64, so why
does it fail for you Gwenore?

Andreas

2003-11-13  Andreas Jaeger  <aj@suse.de>

	* math/libm-test.inc (ceil_test): Test ceil (0.25).

============================================================
Index: math/libm-test.inc
--- math/libm-test.inc	22 Mar 2003 06:13:35 -0000	1.51
+++ math/libm-test.inc	13 Nov 2003 06:09:46 -0000
@@ -1625,6 +1625,7 @@ ceil_test (void)
 
   TEST_f_f (ceil, M_PIl, 4.0);
   TEST_f_f (ceil, -M_PIl, -3.0);
+  TEST_f_f (ceil, 0.25, 1.0);
 
   END (ceil);
 }

-- 
 Andreas Jaeger, aj@suse.de, http://www.suse.de/~aj
  SuSE Linux AG, Deutschherrnstr. 15-19, 90429 Nürnberg, Germany
   GPG fingerprint = 93A3 365E CE47 B889 DF7F  FED1 389A 563C C272 A126

Attachment: pgp00000.pgp
Description: PGP signature