[PATCH] D106753: ConvertUTF: Created wrapper convertUTF32ToUTF8String

Marcus Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 28 18:23:01 PDT 2021


MarcusJohnson91 added a comment.

The problem seems to be in the conversion function expecting strings to be a multiple of 4 bytes, which doesn't hold up with the way ArrayRef stores things as char that is casted to char32_t, when using ASCII values like in the look of disapproval emoji, having an underscore in the middle.

But removing the assert and early return result in even more errors.

changing the input string to remove the underscore also fails, i'm out of ideas.


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

https://reviews.llvm.org/D106753



More information about the llvm-commits mailing list