Page MenuHomeFreeBSD
Differential D28786

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

Authored by arichardson on Feb 19 2021, 12:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 19, 9:11 PM
Unknown Object (File)
Thu, Apr 18, 4:41 PM
Unknown Object (File)
Thu, Apr 18, 4:39 PM
Unknown Object (File)
Thu, Apr 18, 4:38 PM
Unknown Object (File)
Wed, Apr 17, 11:39 PM
Unknown Object (File)
Mar 10 2024, 12:54 AM
Unknown Object (File)
Feb 8 2024, 1:22 AM
Unknown Object (File)
Feb 2 2024, 2:38 AM
View All 44 Files
Subscribers
imp

Details

Reviewers
dim
lwhsu
Commits
rGfe173c91b041: lib/msun: Avoid FE_INEXACT for x86 log2l/log10l
rG221622ec0c8e: lib/msun: Avoid FE_INEXACT for x86 log2l/log10l
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

arichardson created this revision.Feb 19 2021, 12:56 PM
Herald added a subscriber: imp. · View Herald TranscriptFeb 19 2021, 12:56 PM
arichardson requested review of this revision.Feb 19 2021, 12:56 PM
Harbormaster completed remote builds in B37184: Diff 84255.Feb 19 2021, 12:57 PM
arichardson mentioned this in D28798: Convert the msun tests to ATF.Feb 19 2021, 5:32 PM
arichardson updated this revision to Diff 84488.Feb 22 2021, 5:59 PM

Simpler approach

Harbormaster completed remote builds in B37280: Diff 84488.Feb 22 2021, 5:59 PM
arichardson edited the summary of this revision. (Show Details)Feb 22 2021, 6:00 PM
arichardson added a reviewer: lwhsu.
arichardson added a child revision: D28798: Convert the msun tests to ATF.
dim accepted this revision.Mar 5 2021, 8:48 PM

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
Closed by commit rG221622ec0c8e: lib/msun: Avoid FE_INEXACT for x86 log2l/log10l (authored by arichardson). · Explain WhyMar 8 2021, 9:44 AM
This revision was automatically updated to reflect the committed changes.
arichardson added a commit: rG221622ec0c8e: lib/msun: Avoid FE_INEXACT for x86 log2l/log10l.
arichardson mentioned this in rG133bc645077d: Convert the msun tests to ATF.Mar 22 2021, 12:00 PM
arichardson added a commit: rGfe173c91b041: lib/msun: Avoid FE_INEXACT for x86 log2l/log10l.Apr 22 2021, 11:09 AM
arichardson mentioned this in rGb69904b8916c: Convert the msun tests to ATF.

Revision Contents
Changeset List

PathSize
lib/
msun/
ld128/
18 lines
ld80/
9 lines

Diff 85319

View Options

lib/msun/ld128/s_logl.c

Loading...
View Options

lib/msun/ld80/s_logl.c

Loading...