[PATCH] D62957: [yaml2obj/obj2yaml] - Make RawContentSection::Content and RawContentSection::Size optional

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 6 07:38:19 PDT 2019


grimar created this revision.
grimar added a reviewer: jhenderson.
Herald added a subscriber: jakehehrlich.

This is a follow-up for D62809 <https://reviews.llvm.org/D62809>.

`Content` and `Size` fields should be optional as was discussed in comments
of the D62809 <https://reviews.llvm.org/D62809>'s thread. With that, we can describe a specific string table and
symbol table sections in a more correct way and also show appropriate errors.

The patch adds lots of test cases where the behavior is described in details.


https://reviews.llvm.org/D62957

Files:
  include/llvm/ObjectYAML/ELFYAML.h
  lib/ObjectYAML/ELFYAML.cpp
  test/Object/obj2yaml.test
  test/tools/obj2yaml/elf-shinfo.yaml
  test/tools/yaml2obj/section-size-content.yaml
  test/tools/yaml2obj/strtab-implicit-sections-size-content.yaml
  test/tools/yaml2obj/symtab-implicit-sections-size-content.yaml
  tools/obj2yaml/elf2yaml.cpp
  tools/yaml2obj/yaml2elf.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D62957.203362.patch
Type: text/x-patch
Size: 23105 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190606/de7c7d86/attachment-0001.bin>


More information about the llvm-commits mailing list