[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