[libcxx-commits] [PATCH] D97365: [libc++] [C++2b] [P1682] Add to_underlying.

[Marek Kurdej via Phabricator via libcxx-commits]

curdeius marked an inline comment as done.
curdeius added inline comments.


================
Comment at: libcxx/test/std/utilities/utility/utility.underlying/to_underlying.pass.cpp:23
+enum class e_ushort : unsigned short { d = 10, e = 100, f = 200 };
+enum class e_longlong : long long {
+  low = std::numeric_limits<long long>::min(),
----------------
Mordante wrote:
> Maybe interesting to also test these bounds for an plain `enum`.
Ok, will do.


================
Comment at: libcxx/test/std/utilities/utility/utility.underlying/to_underlying.pass.cpp:27
+};
+enum e_non_class { enum_a = 10, enum_b = 11, enum_c = 12 };
+
----------------
Mordante wrote:
> This isn't used in the tests below.
It is in the last 3 assertions.


================
Comment at: libcxx/utils/generate_feature_test_macro_components.py:599
+    "name": "__cpp_lib_to_underlying",
+    "values": { "c++2b": 202102 },
+    "headers": ["utility"],
----------------
Mordante wrote:
> The paper has no number. Is it certain this will be the final value?
I'll wait for the draft to be updated before committing I think to confirm the value.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97365