[PATCH] D36255: Integrate manifest merging library into LLD.

[Zachary Turner via Phabricator via llvm-commits]

zturner added inline comments.


================
Comment at: lld/test/lit.cfg:270
+# libxml2 to merge manifests.
+if (lit.util.which('cvtres', config.environment['PATH'])) or (config.llvm_libxml2_enabled == "1"):
+    config.available_features.add('manifest_tool')
----------------
Can you line-break here?  This goes past 80.  Also, why are we checking for `cvtres` instead of `mt`?  The comment makes sense, but why not just check for `mt` directly?


https://reviews.llvm.org/D36255