[libcxx-commits] [PATCH] D146398: [libcxx] Fix using std::wcout/wcin on Windows with streams configured in wide mode

[Mark de Wever via Phabricator via libcxx-commits]

Mordante accepted this revision.
Mordante added a comment.
This revision is now accepted and ready to land.

LGTM modulo 2 nits.



================
Comment at: libcxx/docs/UsingLibcxx.rst:565
+the sequence of bytes read from, or written to, these streams is interpreted
+as a sequence of littleendian ``wchar_t`` elements. Thus, use of
+``std::cout``, ``std::cerr``, or ``std::cin`` with streams in Unicode mode
----------------



================
Comment at: libcxx/test/std/input.output/iostream.objects/wide.stream.objects/wcerr-wide-mode.sh.cpp:13
+
+// XFAIL: no-wide-characters
+// REQUIRES: target={{.+}}-windows-{{.+}}
----------------
These tests use wide character types internally so I don't think we need to test they really fail.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146398