[PATCH] D149439: [yaml2obj] Add support fot structured section data.

Jacek Caban via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 1 06:51:23 PDT 2023


jacek updated this revision to Diff 527401.
jacek added a comment.

Thanks for review. The new version uses StructuredData instead of Data and does not allow combining StructuredData and SectionData.

My original thinking was that while SectionData is there for backward compatibility, it's becoming redundant as one may use Data: -Binary: instead. A more explicit name sounds good to me too.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D149439/new/

https://reviews.llvm.org/D149439

Files:
  llvm/include/llvm/ObjectYAML/COFFYAML.h
  llvm/lib/ObjectYAML/COFFEmitter.cpp
  llvm/lib/ObjectYAML/COFFYAML.cpp
  llvm/test/tools/yaml2obj/COFF/mixed-data.yaml
  llvm/test/tools/yaml2obj/COFF/structured-data.yaml

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D149439.527401.patch
Type: text/x-patch
Size: 6506 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230601/1f76be4a/attachment-0001.bin>


More information about the llvm-commits mailing list