Not using builtins doesn't always imply worse code,
but for e. g. isinf, this is 30%+ faster.
Before:
name time/op
BM_isinf 2.14ns ± 2%
After:
name time/op
BM_isinf 1.33ns ± 2%
Differential D88854
[libc++] Use builtins in more math.h functions. ClosedPublic Authored by TokarIP on Oct 5 2020, 2:27 PM.
Details
Summary Not using builtins doesn't always imply worse code, Before: After:
Diff Detail
Event TimelineComment Actions Can you re-upload with arc diff to trigger CI? This LGTM if CI passes. And indeed, that would supersede https://reviews.llvm.org/D96121. This revision is now accepted and ready to land.Feb 5 2021, 12:22 PM Closed by commit rGc81d52997ac0: [libc++] Use builtins in more math.h functions. (authored by TokarIP). · Explain WhyFeb 10 2021, 12:29 PM This revision was automatically updated to reflect the committed changes.
Diff 322789 libcxx/include/math.h
|