winv6.3 ins windows 8.1<br><br><div class="gmail_quote">On Wed Oct 22 2014 at 1:34:43 PM Hans Wennborg <<a href="mailto:hans@chromium.org">hans@chromium.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">lgtm2<br>
<br>
================<br>
Comment at: lib/Driver/MSVCToolChain.cpp:<u></u>237<br>
@@ +236,3 @@<br>
+      // It is not necessary to link against Windows SDK 7.x when targeting ARM.<br>
+      libPath.clear();<br>
+      return false;<br>
----------------<br>
Clearing this seems redundant?<br>
<br>
================<br>
Comment at: lib/Driver/MSVCToolChain.cpp:<u></u>243<br>
@@ +242,3 @@<br>
+  } else {<br>
+    // Windows SDK 8.x installs libraries in an folder whose names depend on<br>
+    // the version of the OS you're targeting.  By default choose the newest,<br>
----------------<br>
nit: s/an/a/, s/names/name/ ?<br>
<br>
================<br>
Comment at: lib/Driver/MSVCToolChain.cpp:<u></u>247<br>
@@ +246,3 @@<br>
+    // the SDK on.<br>
+    char *tests[] = {"winv6.3", "win8", "win7"};<br>
+    bool found = false;<br>
----------------<br>
is winv6.3 newer than win8? (i have no idea, just noticed the number is lower)<br>
<br>
<a href="http://reviews.llvm.org/D5873" target="_blank">http://reviews.llvm.org/D5873</a><br>
<br>
<br>
</blockquote></div>