[PATCH] D55308: Implement the second part of P0482 (char8_t)
Marshall Clow via Phabricator
reviews at reviews.llvm.org
Wed Dec 5 14:52:39 PST 2018
mclow.lists marked 3 inline comments as done.
mclow.lists added inline comments.
================
Comment at: test/std/strings/basic.string.hash/enabled_hashes.pass.cpp:26
test_hash_enabled_for_type<std::wstring>();
+#if defined(__cpp_lib_char8_t) && __cpp_lib_char8_t >= 201811L
+ test_hash_enabled_for_type<std::u8string>();
----------------
ldionne wrote:
> Why would the value ever be less than `201811L`? Genuinely asking.
No, but it could get bigger (if we add other `char8_t` stuff, and then the test for the new stuff would be:
`#if defined(__cpp_lib_char8_t) && __cpp_lib_char8_t >= 201903L`, so I just used the same pattern.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55308/new/
https://reviews.llvm.org/D55308
More information about the libcxx-commits
mailing list