[PATCH] D35753: llvm-mt: implement simple merging of manifests, not factoring namespaces.

Eric Beckmann via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 24 16:15:50 PDT 2017


ecbeckmann updated this revision to Diff 107988.
ecbeckmann marked 4 inline comments as done.
ecbeckmann added a comment.

Don't conditionally define destructor, remove global.


https://reviews.llvm.org/D35753

Files:
  llvm/include/llvm/Support/WindowsManifestMerger.h
  llvm/lib/Support/WindowsManifestMerger.cpp
  llvm/test/tools/llvm-mt/Inputs/additional.manifest
  llvm/test/tools/llvm-mt/Inputs/conflicting.manifest
  llvm/test/tools/llvm-mt/Inputs/test_manifest.manifest
  llvm/test/tools/llvm-mt/conflicting.test
  llvm/test/tools/llvm-mt/simple_merge.test
  llvm/tools/llvm-mt/llvm-mt.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35753.107988.patch
Type: text/x-patch
Size: 12104 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170724/2e107ecd/attachment.bin>


More information about the llvm-commits mailing list