<div dir="ltr">On Tue, Aug 20, 2013 at 2:04 PM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi whunt,<br>
<br>
This makes a difference on Windows, where e.g. clang-cl might actually be called as CLANG-CL.<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D1450" target="_blank">http://llvm-reviews.chandlerc.com/D1450</a><br>
<br>
Files:<br>
  tools/driver/driver.cpp<br>
<br>
Index: tools/driver/driver.cpp<br>
===================================================================<br>
--- tools/driver/driver.cpp<br>
+++ tools/driver/driver.cpp<br>
@@ -226,6 +226,8 @@<br>
     { "++",        "--driver-mode=g++" },<br>
   };<br>
   std::string ProgName(llvm::sys::path::stem(ArgVector[0]));<br>
+  for (size_t i = 0, e = ProgName.length(); i != e; ++i)<br>
+    ProgName[i] = toLowercase(ProgName[i]);<br></blockquote><div><br></div><div>This is modern C++, we can't use for loops, what are we, farmers!?</div><div><div>std::transform(ProgName.begin(), ProgName.end(), ProgName.begin(), ::toLowerCase);</div>
</div><div><br></div><div>;)</div><div><br></div><div>Also, should we do this transform on non-Windows platforms?</div></div></div></div>