[libcxx-commits] [libcxx] [libc++] Remove _LIBCPP_SHORT_WCHAR (PR #207562)

[Louis Dionne via libcxx-commits]

https://github.com/ldionne commented:

I think this patch needs to come with some documentation change to reflect what we've said in https://github.com/llvm/llvm-project/issues/195149. I also don't really feel like that discussion has a clear conclusion.

In https://github.com/llvm/llvm-project/issues/195149#issuecomment-4833887783, Aaron pointed out that we have additional UB in the library (and I think he's right). Before we actually make a move like this, perhaps we should close that loop?

https://github.com/llvm/llvm-project/pull/207562