<div dir="ltr">Not sure if my entire email got sent.  But I was saying there are multiple APIs like for UTF16 and UTF32, various directions of conversion (8->16 and 16->8), etc.  So if it's going to be done for one function it makes sense to do it everywhere.  then, there's the issue of sharing code between the various overloads that will inevitably arise if we add a second overload of each function.<div><br></div><div>I don't think it makes sense to block this fix on a largely cosmetic API fix</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, May 9, 2017 at 12:34 PM Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Yes, bot is red.  Even still, the unicode api fix is rather involved (I actually tried) and it doesn't seem worth blocking this on that since it's largely unrelated and there are a lot of boring details to handle like which subset of apis to do it for, how to res<div><br><div class="gmail_quote"><div dir="ltr">On Tue, May 9, 2017 at 12:32 PM Rui Ueyama via Phabricator <<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ruiu added a comment.<br>
<br>
Do you mean the bot is still red? If I broke something I'd first try to roll it back if the error is not just a simple mistake but needs another round of code review.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D33014" rel="noreferrer" target="_blank">https://reviews.llvm.org/D33014</a><br>
<br>
<br>
<br>
</blockquote></div></div></div></blockquote></div>