[PATCH] D107202: ConvertUTF: convertUTF32ToUTF8String

Marcus Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 4 02:46:25 PDT 2021


MarcusJohnson91 added a comment.

In D107202#2924561 <https://reviews.llvm.org/D107202#2924561>, @efriedma wrote:

> The the way the math is written now, for "Example", we allocate UNI_MAX_UTF8_BYTES_PER_CODE_POINT * sizeof(UTF32) * 7 = 112 bytes.

Alright, I'm gonna give it a try and re-run the tests


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

https://reviews.llvm.org/D107202



More information about the llvm-commits mailing list