[PATCH] D114342: ConvertUTF, new wrapper API

Wed Mar 23 19:02:58 PDT 2022

MarcusJohnson91 added a comment.

In D114342#3402219 <https://reviews.llvm.org/D114342#3402219>, @aaron.ballman wrote:

> In D114342#3401916 <https://reviews.llvm.org/D114342#3401916>, @cor3ntin wrote:
>
>> @MarcusJohnson91 The failure looks unrelated. The test looks good, I think we can try again. I'd rather wait for @aaron.ballman in case you need to revert again though!
>
> Agreed that the precommit CI failure looks unrelated and that the test seems reasonable. I've landed again in d14ccbc2e88d2cebd27cdd829879ded10ba0c9ea <https://reviews.llvm.org/rGd14ccbc2e88d2cebd27cdd829879ded10ba0c9ea> and we'll see how happy the build bots are.

Seems like they were happy after all? 🤞

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

https://reviews.llvm.org/D114342