[PATCH] D67445: [yaml2obj/ObjectYAML] - Cleanup the error reporting API, add custom errors handlers.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 11 07:12:50 PDT 2019


grimar created this revision.
grimar added reviewers: jhenderson, MaskRay.
Herald added subscribers: aheejin, sbc100.

This is a continuation of the YAML library error reporting
refactoring/improvement and the idea by itself was mentioned
in the following thread:
https://reviews.llvm.org/D67182?id=218714#inline-603404

This performs a cleanup of all object emitters in the library.
It allows using the custom one provided by the caller.

One of the nice things is that each tool can now print its tool name,
e.g: "yaml2obj: error: <text>"

Also, the code became a bit simpler.


https://reviews.llvm.org/D67445

Files:
  include/llvm/ObjectYAML/yaml2obj.h
  lib/ObjectYAML/COFFEmitter.cpp
  lib/ObjectYAML/ELFEmitter.cpp
  lib/ObjectYAML/MachOEmitter.cpp
  lib/ObjectYAML/MinidumpEmitter.cpp
  lib/ObjectYAML/WasmEmitter.cpp
  lib/ObjectYAML/yaml2obj.cpp
  test/ObjectYAML/wasm/invalid_section_order.yaml
  test/tools/yaml2obj/coff-invalid-alignment.test
  test/tools/yaml2obj/dynsymtab-implicit-sections-size-content.yaml
  test/tools/yaml2obj/empty-or-invalid-doc.yaml
  test/tools/yaml2obj/invalid-docnum.test
  test/tools/yaml2obj/invalid_output_file.test
  test/tools/yaml2obj/missing_document_tag.yaml
  test/tools/yaml2obj/multi-doc.test
  test/tools/yaml2obj/reloc-sec-info.yaml
  test/tools/yaml2obj/section-size-content.yaml
  tools/llvm-ifs/llvm-ifs.cpp
  tools/yaml2obj/yaml2obj.cpp
  unittests/ObjectYAML/MinidumpYAMLTest.cpp
  unittests/ObjectYAML/YAML2ObjTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D67445.219706.patch
Type: text/x-patch
Size: 53974 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190911/c913baf4/attachment.bin>


More information about the llvm-commits mailing list