[libcxx-commits] [PATCH] D127650: [libc++] Remove macros for IBM compiler

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Jun 13 15:04:29 PDT 2022

ldionne added subscribers: SeanP, hubert.reinterpretcast, daltenty.
ldionne marked an inline comment as done.
ldionne added a comment.

In D127650#3578528 <https://reviews.llvm.org/D127650#3578528>, @Mordante wrote:

> Nice cleanup LGTM! I think it would be good to add some of the IBM people as reviewer too.

We discussed this with @SeanP, but adding others as well.

Comment at: libcxx/include/__config:516
Mordante wrote:
> Can `_LIBCPP_HAS_NO_UNICODE_CHARS` be set in CMake by users? If not this seems to be the only place where `_LIBCPP_HAS_NO_UNICODE_CHARS` is defined.
> In that case do you want to make a followup patch to remove this, or do you want me to do it?
Oh, wow, that's freakin' great. I've been meaning to clean this up for a very long time, but I didn't know how to. I'll actually fold the removal into this patch since it's directly tied to `_LIBCPP_COMPILER_IBM`.

  rG LLVM Github Monorepo



More information about the libcxx-commits mailing list