[libcxx-commits] [PATCH] D148542: [libc++] cuchar redeclares ::mbstate_t when it's in its own clang module

Ian Anderson via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Apr 20 10:17:40 PDT 2023


iana added a comment.

The sanitizer and aix test failures don't look related to this change, I get the same ones in an unrelated review too.



================
Comment at: libcxx/include/uchar.h:59
+// Get the declaration for size_t.
 # include <stddef.h>
+
----------------
This should maybe be
```
# define __need_size_t
# include_next <stddef.h>
```
Or maybe that's a bit too stringent.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148542



More information about the libcxx-commits mailing list