Use std::nextafter() instead of std::nexttoward() in midpoint tests.
In the context of this test, this should not cause any difference.
Since nexttowardl() is not implemented on NetBSD 8, the latter function
combined with 'long double' type caused test failure. nextafterl() does
not have this problem.
Details
Details
- Reviewers
mclow.lists EricWF krytarowski - Commits
- rZORG6fe53f1dfaff: [libc++] [test] Use std::nextafter() instead of std::nexttoward()
rZORG0c8b9f68745b: [libc++] [test] Use std::nextafter() instead of std::nexttoward()
rG6fe53f1dfaff: [libc++] [test] Use std::nextafter() instead of std::nexttoward()
rG0c8b9f68745b: [libc++] [test] Use std::nextafter() instead of std::nexttoward()
rCXX360673: [libc++] [test] Use std::nextafter() instead of std::nexttoward()
rG87ae6bf80b4d: [libc++] [test] Use std::nextafter() instead of std::nexttoward()
rL360673: [libc++] [test] Use std::nextafter() instead of std::nexttoward()
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
I'm ok with this change, but I'm wondering - what is the reason that NetBSD doesn't implement nextafter?
Comment Actions
It's nexttowardl() that's missing in nb8. I don't really know why, probably nobody needed it at the point (I guess because it's no different than nextafterl()), it was put as TODO and forgotten. I've added it to nb9 but the backport to nb8 is currently blocked on some ABI stability debate. Until that's resolved, and since it doesn't seem to make any difference here, the change makes it possible to make tests pass again.