[PATCH] D37240: Fix crbug 759265 by suppressing llvm mt warnings.
Eric Beckmann via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 1 14:12:17 PDT 2017
ecbeckmann added inline comments.
================
Comment at: lld/COFF/DriverUtils.cpp:431
+ std::string Msg = toString(InternalOutputBufferOrError.takeError());
+ if (Msg != "no libxml2")
+ fatal("error with internal manifest tool:" + Msg);
----------------
hans wrote:
> I suppose this will solve the immediate problem, but it still looks a bit strange.
>
> I'm not very familiar with this code, but don't we know beforehand if the internal mt tool is available and expected to work?
>
> As Rui pointed out, why can't we just do:
>
> ```
> If the internal mt is present:
> If merge fails for some reason:
> Report that error and abort
> Otherwise:
> Invoke external mt command, and if it fails, abort
> ```
The only way we can tell if it will work is if we see if libxml2 is enabled, but as I said before that is an implementation detail internal to the library that users should not need to know about. I suppose we can add a cmake variable that has the same value as the libxml2 flag.
https://reviews.llvm.org/D37240
More information about the llvm-commits
mailing list