This is an archive of the discontinued LLVM Phabricator instance.

[libc][math] Added atanhf function.
ClosedPublic

Authored by orex on Aug 28 2022, 11:06 AM.

Details

Summary

Performance by core-math (core-math/glibc 2.31/current llvm-14):
10.845/43.174/13.467

The review is done on top of D132809.

Diff Detail

Event Timeline

orex created this revision.Aug 28 2022, 11:06 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptAug 28 2022, 11:06 AM
orex requested review of this revision.Aug 28 2022, 11:06 AM
zimmermann6 accepted this revision.Aug 29 2022, 1:47 AM

ok for me. I get slightly better figures for llvm-libc:

zimmerma@biscotte:~/svn/core-math$ LIBM=/localdisk/zimmerma/llvm-project/build/projects/libc/lib/libllvmlibc.a CORE_MATH_PERF_MODE=rdtsc ./perf.sh atanhf
GNU libc version: 2.34
GNU libc release: stable
23.547
70.432
20.065

This is on a AMD EPYC 7282.

This revision is now accepted and ready to land.Aug 29 2022, 1:47 AM
orex added a comment.Aug 29 2022, 2:22 AM

Thank you, Paul! I'll land the revision after Tue review.

lntue accepted this revision.Aug 29 2022, 1:16 PM
lntue added inline comments.
libc/src/math/generic/atanhf.cpp
22

|x| >= 1.0 ?

orex updated this revision to Diff 456460.Aug 29 2022, 2:24 PM

Fixed comments.

orex marked an inline comment as done.Aug 29 2022, 2:24 PM
orex updated this revision to Diff 456462.Aug 29 2022, 2:26 PM

Fixed entrypoint.

orex updated this revision to Diff 456755.Aug 30 2022, 12:29 PM

Review fixes.

lntue accepted this revision.Aug 30 2022, 12:39 PM
lntue added inline comments.Aug 30 2022, 12:40 PM
libc/src/math/generic/CMakeLists.txt
1235

Update this dependency.

This revision was landed with ongoing or failed builds.Aug 30 2022, 1:40 PM
This revision was automatically updated to reflect the committed changes.