majnemer added a subscriber: majnemer.

================
Comment at: cfe/trunk/lib/Driver/MSVCToolChain.cpp:263
@@ +262,3 @@
+  std::sscanf(RegistrySDKVersion.c_str(), "v%d.", &Major);
+  if (Major > 7 && Major < 10) {
+    // Windows SDK 8.x installs libraries in a folder whose names depend on the
----------------
Isn't this just `Major == 8 || Major == 9` ?

================
Comment at: cfe/trunk/lib/Driver/MSVCToolChain.cpp:278-279
@@ +277,4 @@
+      return false;
+  }
+  else {
+    if (!getWindows10SDKVersion(Path, WindowsSDKIncludeVersion))
----------------
Same line please.


http://reviews.llvm.org/D12832



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to