[PATCH] D49398: clang-cl: Postpone Wmsvc-not-found emission until link.exe gets used.

[Zachary Turner via Phabricator via cfe-commits]

zturner added inline comments.


================
Comment at: lib/Driver/ToolChains/MSVC.cpp:467-468
   if (Linker.equals_lower("link")) {
+    if (!TC.FoundMSVCInstall())
+      C.getDriver().Diag(clang::diag::warn_drv_msvc_not_found);
+
----------------
It looks like it's possible for this warning to be emitted even when `FindVisualStudioExecutable` succeeds (after looking in the install location it checks `PATH`).  Would it make more sense to put this check after the call to `FindVisualStudioExecutable`, but only if it fails?


https://reviews.llvm.org/D49398