Page MenuHomeFreeBSD

lib/msun: Avoid FE_INEXACT for x86 log2l/log10l
ClosedPublic

Authored by arichardson on Feb 19 2021, 12:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 3, 1:15 PM
Unknown Object (File)
Oct 5 2024, 5:45 AM
Unknown Object (File)
Oct 2 2024, 6:13 PM
Unknown Object (File)
Oct 2 2024, 2:24 PM
Unknown Object (File)
Oct 2 2024, 1:26 PM
Unknown Object (File)
Sep 29 2024, 9:27 PM
Unknown Object (File)
Sep 29 2024, 2:04 PM
Unknown Object (File)
Sep 8 2024, 10:22 PM
Subscribers

Details

Summary

This fixes tests/lib/msun/logarithm_test after compiling the test with
-fno-builtin (D28577). Adding invln10_lo + invln10_10 results in
FE_INEXACT and the same for the log2l invln2_lo + invln2_hi. This patch
avoids FE_INEXACT by defining a constant and using that.

Test Plan

logarithm_test works for me now.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Although the original authors seem to dislike constant folding by the compiler, it is a good idea here. :)

This revision is now accepted and ready to land.Mar 5 2021, 8:48 PM