abhina.sreeskantharajan marked an inline comment as done.
abhina.sreeskantharajan added a comment.

Please correct me if I'm wrong, I'm not too familiar with icu4c, but I think 
adding support for ICU would be the better long-term solution since it seems to 
allow the same behaviour across different platforms. However, the issue on the 
z/OS platform is that there currently isn't support for this library so iconv 
seems to be the only solution we can use until we do get support. So would an 
alternative be to use iconv only on z/OS (and hopefully this is a temporary 
solution until icu is supported on z/OS) and use icu on all other platforms?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153418

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to