[PATCH] D61086: llvm-cvtres: Make new dupe resource error a bit friendlier

[Nico Weber via Phabricator via llvm-commits]

thakis created this revision.
thakis added a reviewer: mstorsjo.
Herald added subscribers: rupprecht, hiraditya.
Herald added a project: LLVM.

For well-known type IDs, include the name of the type.

To not duplicate the ID->name map, make llvm-readobj call this new
function as well.  It has slightly different output, so this also
requires updating a few tests.


https://reviews.llvm.org/D61086

Files:
  lld/test/COFF/manifestinput.test
  lld/test/COFF/resource.test
  llvm/include/llvm/Object/WindowsResource.h
  llvm/lib/Object/WindowsResource.cpp
  llvm/test/tools/llvm-cvtres/combined.test
  llvm/test/tools/llvm-cvtres/duplicate.test
  llvm/test/tools/llvm-cvtres/object.test
  llvm/test/tools/llvm-readobj/coff-resources.test
  llvm/tools/llvm-cvtres/llvm-cvtres.cpp
  llvm/tools/llvm-readobj/COFFDumper.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D61086.196501.patch
Type: text/x-patch
Size: 12795 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190424/c69bd1aa/attachment.bin>