[libcxx-commits] [libcxx] [libc++] Add std::fpclassify overloads for floating-point. (PR #67913)

via libcxx-commits libcxx-commits at lists.llvm.org
Sun Oct 1 06:28:38 PDT 2023


https://github.com/philnik777 requested changes to this pull request.

The implementation looks good, but this needs tests. Specifically,
- there should be a regression test to make sure an implicit conversion works now
- please make sure that we have test coverage for the `_LIBCPP_NODISCARD_EXT`s
- please make sure that we have test for the `noexcept`ness of the overloads.
You should be able to find tests for this under `libcxx/test/libcxx/{diagnostics,numerics/c.math}` if they exist.

https://github.com/llvm/llvm-project/pull/67913


More information about the libcxx-commits mailing list