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

	F161383826: D28786.id84255.diff
	Fri, Jul 3, 7:35 AM

	F161368132: D28786.id.diff
	Fri, Jul 3, 5:10 AM

	F161363197: D28786.id84488.diff
	Fri, Jul 3, 4:16 AM

	F161324211: D28786.id84255.diff
	Thu, Jul 2, 7:58 PM

	Unknown Object (File)
	Thu, Jul 2, 7:35 AM

	Unknown Object (File)
	Fri, Jun 26, 4:02 PM

	Unknown Object (File)
	Thu, Jun 25, 12:37 AM

	Unknown Object (File)
	Thu, Jun 18, 5:30 PM

Subscribers

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

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 37280
Build 34169: arc lint + arc unit

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

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.

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

Path

Size

lib/

msun/

ld128/

18 lines

ld80/

9 lines

Diff 84488

lib/msun/ld128/s_logl.c

Loading...

lib/msun/ld80/s_logl.c

Loading...