Hi everyone,
Sorry for the late reply, I did not have time to look into ICU support until recently. I have updated my old PR which had iconv support, and extended it with very basic ICU support. With my current solution, we check for icu support first, and if that is not available, we then check for iconv support.
This PR can be found here: Create a CharSetConverter class with both iconv and icu support by abhina-sree · Pull Request #74516 · llvm/llvm-project · GitHub
I was not able to test on Windows, so would greatly appreciate your feedback on if this path forward meets the needs of all platforms.
Thanks for your patience,
Abhina