[PATCH] D107202: ConvertUTF: convertUTF32ToUTF8String

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

In D106753#2917762 <https://reviews.llvm.org/D106753#2917762>, @efriedma wrote:

> As far as I can tell, the lastest version of the diff you uploaded still has the following issues that haven't been addressed:
>
> 1. The BOM handling is actually actively a problem if you're planning to use the interface to interpret wprintf format strings. We don't want to byteswap `L"\uFFFE%s"` or something like that.
> 2. `SrcBytes.size() * UNI_MAX_UTF8_BYTES_PER_CODE_POINT + 1` seems like way too much memory.


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

https://reviews.llvm.org/D107202