This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH 1/2] sparc: remove ceil, floor, trunc sparc specific implementations
From: Aurelien Jarno <aurelien@aurel32.net>
Date: Mon, 1 Aug 2016 23:50:25 +0200
> On 2016-08-01 14:44, David Miller wrote:
>> From: Aurelien Jarno <aurelien@aurel32.net>
>> Date: Mon, 1 Aug 2016 15:24:22 +0200
>>
>> > The ceil, floor and trunc functions on sparc do not fully follow the
>> > standard and trigger an inexact exception when presented a value which
>> > is not an integer. Since glibc 2.24 this causes a few tests to fail,
>> > for instance:
>>
>> Carlos, this specific patch fixes most of the math failures on sparc,
>> could we get this into the release?
>
> I think you should ask Adhemerval instead.
>
>> Aurelien, it looks like we have the same exact issue with nearbyint on
>> sparc, right?
>
> I don't see the issue on nearbyint here. What is the issue exactly?
Maybe only the vis3 variant shows the problem:
Failure: nearbyint (sNaN): Exception "Invalid operation" not set
Failure: nearbyint (-sNaN): Exception "Invalid operation" not set
Failure: nearbyint_downward (sNaN): Exception "Invalid operation" not set
Failure: nearbyint_downward (-sNaN): Exception "Invalid operation" not set
Failure: nearbyint_towardzero (sNaN): Exception "Invalid operation" not set
Failure: nearbyint_towardzero (-sNaN): Exception "Invalid operation" not set
Failure: nearbyint_upward (sNaN): Exception "Invalid operation" not set
Failure: nearbyint_upward (-sNaN): Exception "Invalid operation" not set
>> And that leaves the fdim failures, which are probably a similar
>> problem.
>
> Yes, I am currently working on the fdim issue, to see if we should apply
> the same strategy or fix it.
Ok.