[libcxx-commits] [libcxx] [libc++][z/OS] Need to define _LIBCPP_HAS_UNICODE to 0 for EBCDIC (PR #119244)

[Zibi Sarbinowski via libcxx-commits]

zibi2 wrote:

> Supporting EBCDIC and z/OS requires a lot more changes than just this unless I am mistaken. I know you folks tried upstreaming a lot of that support a few years ago but AFAICT we never got consensus for moving forward with those changes, which were quite involved. AIX doesn't have the same challenges and we're happy to support it upstream.
> 
> I would like to have a broader discussion about EBCDIC support in libc++ before we start cherry-picking small changes like this into our codebase -- that ask is consistent with what we do for other platforms that are not "officially supported".

Louis, this change is the result of relatively new common changes which broke us and we are trying very hard to absorb ALL changes and correct them as they come to keep all libc++ tests clean. It does help us to stay current and do not degrade.  I hope you will accept this and approve it since this is not part of the larger changes we need to discuss again. 

FYI, @perry-ca is going to write document likely RFC about the changes we need to support libc++ on z/OS and that is not just EBCDIC related.


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