[libcxx-commits] [PATCH] D60951: [libc++][test] Add test coverage for codecvt<char(16|32)_t, char8_t, mbstate_t>

Casey Carter via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri May 8 06:24:25 PDT 2020


CaseyCarter closed this revision.
CaseyCarter added a comment.

In D60951#2026475 <https://reviews.llvm.org/D60951#2026475>, @ldionne wrote:

> To be clear, I'm trying to move the test suite towards a generic conformance test suite for the Standard, as opposed to a libc++ test suite that happens to be usable by other Standard Libraries. This change highlights that philosophy.


Excellent. I'll try to contribute more tests in the future.

This was merged as https://github.com/llvm/llvm-project/commit/197f185274567f56c8824a63cb2d8b1a0c76b29a. (I didn't notice until too late that I didn't have the Differential Revision in the commit log, so closing by hand like a barbarian.)


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

https://reviews.llvm.org/D60951





More information about the libcxx-commits mailing list