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

Jacek Caban via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 4 15:34:59 PDT 2023


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

I wasn't sure about capitalization when I wrote it. I still don't have a strong preference, but decided to change it to UInt32 after reconsideration.

A signed variant seems unlikely to be important in COFF context. I've been thinking about something like Zeroes (filling with specified number of zeroes) as another possible extension.


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

https://reviews.llvm.org/D149439

Files:
  llvm/docs/yaml2obj.rst
  llvm/include/llvm/ObjectYAML/COFFYAML.h
  llvm/lib/ObjectYAML/COFFEmitter.cpp
  llvm/lib/ObjectYAML/COFFYAML.cpp
  llvm/test/tools/llvm-readobj/arm64ec-chpe.yaml
  llvm/test/tools/yaml2obj/COFF/invalid-raw-data.yaml
  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.537185.patch
Type: text/x-patch
Size: 9897 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230704/eae02327/attachment.bin>


More information about the llvm-commits mailing list