[PATCH] D153418: Adding iconv support to CharSetConverter class
Abhina Sree via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jun 22 06:32:14 PDT 2023
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
More information about the cfe-commits
mailing list