[libcxx-commits] [PATCH] D90222: [1/N] [libcxx] Implement c++2a char8_t input/output of std::filesystem::path
Martin Storsjö via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Thu Dec 3 08:35:09 PST 2020
mstorsjo added inline comments.
================
Comment at: libcxx/include/filesystem:1006
+#ifndef _LIBCPP_NO_HAS_CHAR8_T
+ _LIBCPP_INLINE_VISIBILITY _VSTD::u8string u8string() const { return _VSTD::u8string(__pn_.begin(), __pn_.end()); }
+#else
----------------
ldionne wrote:
> That's an ABI break in C++20. Do we think it's reasonable?
>
> I think we at least need to add it to the release notes.
This method is marked with the ` _LIBCPP_INLINE_VISIBILITY` attribute, which afaik is supposed to exclude it from the ABI altogether - exactly for this reason?
The fact that it's a source compat break, and whether to worry about it or not, is acknowledged and discussed in P1423.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D90222/new/
https://reviews.llvm.org/D90222
More information about the libcxx-commits
mailing list