[libcxx-commits] [PATCH] D77505: [libc++] Implement <numbers>
Raul Tambre via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Sat Jun 6 04:14:40 PDT 2020
tambre added a comment.
I've addressed the comments.
Comment at: libcxx/include/numbers:103
+template <class T>
+concept floating_point = std::is_floating_point_v<T>;
> I don't see that concept defined inside `<numbers>` in http://wg21.link/p0631 -- should this be kept as an implementation detail instead?
> Or was it moved there by another paper? In all cases, if that is public according to the Standard, please add tests for it.
Yes, this is an implementation detail. I've prefixed it with two underscores now.
Technically this should be `std::floating_point` from the concepts library, but libc++ doesn't have a full implementation of that yet. Besides, it seems a bit heavy to pull in `<concepts>` just for one thing, when it's simple enough to define here.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the libcxx-commits