[libcxx-commits] [PATCH] D59937: Added implementation of expint [sf.cmath]

Andre Brand via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon May 13 05:33:52 PDT 2019


thebrandre updated this revision to Diff 199252.
thebrandre edited the summary of this revision.
thebrandre added a comment.

- The implementation is inline for the same reason as mentioned in  D60869 <https://reviews.llvm.org/D60869>.
- Moved the tests for consistency with the latest changes in cmath (lerp).
- Simplification of the tests: now using only one set of reference values generated by boost with multiprecision floats (256bit rounded to 128bit). I am pretty optimistic that every digit is correct, so that the comparison to other implementations no longer serves any purpose.
- Rebased against current master branch.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59937/new/

https://reviews.llvm.org/D59937

Files:
  libcxx/include/cmath
  libcxx/test/std/numerics/c.math/c.math.expint/expint_bounds.pass.cpp
  libcxx/test/std/numerics/c.math/c.math.expint/expint_comparisons.pass.cpp
  libcxx/test/std/numerics/c.math/c.math.expint/expint_limits.pass.cpp
  libcxx/test/std/numerics/c.math/cmath.pass.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D59937.199252.patch
Type: text/x-patch
Size: 32936 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20190513/f0e09dbd/attachment-0001.bin>


More information about the libcxx-commits mailing list