<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Mar 8, 2016 at 3:28 PM Cameron <<a href="mailto:cameron@moodycamel.com">cameron@moodycamel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">cameron314 added inline comments.<br>
<br>
================<br>
Comment at: llvm/trunk/lib/Support/CommandLine.cpp:793<br>
@@ +792,3 @@<br>
+  if (!llvm::ConvertUTF8toWide(envVar, wenvVar)) {<br>
+    assert(false);<br>
+    return;<br>
----------------<br>
aaron.ballman wrote:<br>
> Can you turn them into `assert(false && "Message about what went wrong");`?<br>
Will do.<br>
<br>
================<br>
Comment at: llvm/trunk/lib/Support/ConvertUTFWrapper.cpp:190<br>
@@ +189,3 @@<br>
+  const UTF8 *ErrorPtr;<br>
+  if (!ConvertUTF8toWide(sizeof(wchar_t), Source, ResultPtr, ErrorPtr))<br>
+  {<br>
----------------<br>
aaron.ballman wrote:<br>
> You should run clang-format over this as well, the formatting isn't correct (here and elsewhere in the file).<br>
Will do. I was hoping to avoid this since clang-format changes a lot of code that was already present as well. I'll extract just the parts that apply to my patch, I suppose.<br></blockquote><div><br></div><div>1. Find the script git-clang-format in your llvm/tools/clang repo.  </div><div>2. Add its directory to your PATH.  </div><div>3. Check in all your changes so that your patch is a single squashed commit at tip of trunk</div><div>4. run `git clang-format HEAD~1`</div><div><br></div><div>That should format only lines you modified.</div></div></div>