[libcxx-commits] [PATCH] D155185: [libc++] Work around #40363 for stringbuf::str()

[Hans Wennborg via Phabricator via libcxx-commits]

hans added inline comments.


================
Comment at: libcxx/include/sstream:359
+      requires __is_allocator<_SAlloc>::value
+    _LIBCPP_HIDE_FROM_ABI basic_string<char_type, traits_type, _SAlloc> str(const _SAlloc& __sa) const {
+        return basic_string<_CharT, _Traits, _SAlloc>(view(), __sa);
----------------
pfusik wrote:
> hans wrote:
> > Moving this one seems unrelated?
> It **is** related. This overload does not conflict with the pre-C++20 one, so it shouldn't depend on `_LIBCPP_BUILDING_LIBRARY`.
I mean it doesn't seem related to the problem we're working around with _LIBCPP_HIDE_FROM_ABI_SSTREAM, so perhaps it would be better to do it in a separate patch?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155185