[PATCH] D97656: [llvm-objcopy] Initial XCOFF32 support.

Esme Yi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 17 03:49:02 PDT 2021


Esme updated this revision to Diff 373181.
Esme added a comment.

1. Address comment.
2. Add fields of OptionalFileHeader, AuxSymbolEntries, and LineNumInfo, otherwise an error will occur when using the compiled binary as input. (OptionalFileHeader and AuxSymbolEntries are not supported yet, just copy their contents directly.)
3. Add error tests except for the error path in getLineNumberInfo because yaml2obj does not support it.
4. Add a binary test with LineNumInfo and AuxSymbolEntries.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97656

Files:
  llvm/include/llvm/BinaryFormat/XCOFF.h
  llvm/include/llvm/Object/XCOFFObjectFile.h
  llvm/lib/Object/XCOFFObjectFile.cpp
  llvm/test/tools/llvm-objcopy/XCOFF/Inputs/basic32.o
  llvm/test/tools/llvm-objcopy/XCOFF/basic-copy.test
  llvm/test/tools/llvm-objcopy/XCOFF/binary-copy.test
  llvm/test/tools/llvm-objcopy/XCOFF/invalid-section.test
  llvm/tools/llvm-objcopy/CMakeLists.txt
  llvm/tools/llvm-objcopy/ConfigManager.cpp
  llvm/tools/llvm-objcopy/ConfigManager.h
  llvm/tools/llvm-objcopy/MultiFormatConfig.h
  llvm/tools/llvm-objcopy/XCOFF/Object.h
  llvm/tools/llvm-objcopy/XCOFF/Reader.cpp
  llvm/tools/llvm-objcopy/XCOFF/Reader.h
  llvm/tools/llvm-objcopy/XCOFF/Writer.cpp
  llvm/tools/llvm-objcopy/XCOFF/Writer.h
  llvm/tools/llvm-objcopy/XCOFF/XCOFFConfig.h
  llvm/tools/llvm-objcopy/XCOFF/XCOFFObjcopy.cpp
  llvm/tools/llvm-objcopy/XCOFF/XCOFFObjcopy.h
  llvm/tools/llvm-objcopy/llvm-objcopy.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D97656.373181.patch
Type: text/x-patch
Size: 31284 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210917/170a2a4c/attachment.bin>


More information about the llvm-commits mailing list