[PATCH] D36201: Merge manifest namespaces.

[Eric Beckmann via Phabricator via llvm-commits]

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

Avoid static constructor, remove symbols from global namespace, separate real and stub implementations, simplify string compare.


https://reviews.llvm.org/D36201

Files:
  llvm/include/llvm/WindowsManifest/WindowsManifestMerger.h
  llvm/lib/WindowsManifest/WindowsManifestMerger.cpp
  llvm/test/tools/llvm-mt/Inputs/assembly_identity.manifest
  llvm/test/tools/llvm-mt/Inputs/compatibility.manifest
  llvm/test/tools/llvm-mt/Inputs/test_manifest.manifest
  llvm/test/tools/llvm-mt/Inputs/trust_and_identity.manifest
  llvm/test/tools/llvm-mt/Inputs/trust_info.manifest
  llvm/test/tools/llvm-mt/Inputs/windows_settings.manifest
  llvm/test/tools/llvm-mt/big_merge.test
  llvm/test/tools/llvm-mt/simple_merge.test
  llvm/test/tools/llvm-mt/single_file.test
  llvm/tools/llvm-mt/Opts.td
  llvm/tools/llvm-mt/llvm-mt.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D36201.109404.patch
Type: text/x-patch
Size: 40307 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170802/b92a65ac/attachment-0001.bin>