Bug 31818 - math/test-float128-log2p1, math/test-float64x-log2p1, math/test-ldouble-log2p1 fails on ppc64le-power8-nomultiarch
Summary: math/test-float128-log2p1, math/test-float64x-log2p1, math/test-ldouble-log2...
Status: RESOLVED FIXED
Alias: None
Product: glibc
Classification: Unclassified
Component: math (show other bugs)
Version: 2.39
: P2 normal
Target Milestone: ---
Assignee: Not yet assigned to anyone
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-05-30 09:26 UTC by Jeevitha P.
Modified: 2024-06-21 10:18 UTC (History)
3 users (show)

See Also:
Host: powerpc*-*-*
Target:
Build:
Last reconfirmed:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jeevitha P. 2024-05-30 09:26:41 UTC
Log2p1 was implemented in commit id(79c52daf4740b00bef43e4f5a4281b3480459b38)and maximum allowed error was updated for powerpc in the commit (f83e461f1014598a5cb4c89407ce303b9f0bd8ac). We are seeing the tests was failing with "--disable-multi-arch" in power8 LE.
The issue with those test cases is that the expected ULP from "sysdeps/powerpc/fpu/libm-test-ulps" does not match the result. The expected ULP for float128 is 2, but we are getting 3 for Power8.
However, there are no issues on Power9 and Power10 since their output has error of 2 ulp. Can we change the ulps for float128 to 3? Is this a reasonable change.
Comment 1 Joseph Myers 2024-05-30 17:23:31 UTC
It's generally reasonable to update ulps like that (small differences can often result from e.g. compiler choices about whether to use fused operations).
Comment 2 Jeevitha P. 2024-06-21 10:18:07 UTC
This issue has been resolved by changing the ULP value for float128. Issue resolved by the following commit.

commit 71dafdf5f19dd2b0729e4774149944911a405bc6
Author:     Florian Weimer <fweimer@redhat.com>
AuthorDate: Thu Jun 20 11:07:17 2024 +0200
Commit:     Florian Weimer <fweimer@redhat.com>
CommitDate: Thu Jun 20 12:15:31 2024 +0200

    powerpc: Update ulps
    
    Results based on POWER8 and POWER9 machines running
    powerpc64-linux-gnu, with and without --disable-multi-arch.