[PATCH] D37240: Fix crbug 759265 by suppressing llvm mt warnings.

[Eric Beckmann via Phabricator via llvm-commits]

ecbeckmann added inline comments.


================
Comment at: lld/COFF/DriverUtils.cpp:386-387
   if (!HostTriple.isOSWindows())
-    fatal("manifest ignored because no external manifest tool available");
+    fatal("external mt.exe is not available because this is not a Windows "
+          "system");
   // Create the default manifest file as a temporary file.
----------------
ruiu wrote:
> There are Windows systems that don't have mt.exe. :)
You're right, Executor will cover the failure to launch mt.exe, whatever the context.


https://reviews.llvm.org/D37240