[PATCH] D63843: [Object][XCOFF] Add support for 64-bit file header and section header dumping.

[Sean Fertile via Phabricator via llvm-commits]

sfertile updated this revision to Diff 208179.
sfertile marked 2 inline comments as done.
sfertile added a comment.

- Perform address arithmetic on `const char*` before casting to a uintptr_t.
  - Fixup comment. `NumberOfX == STYP_OVRFLO` doesn't make sense as STYP_OVRFLO is a section type. _OVERFLOW_MARKER is the name of the constant representing the overflow marker value.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63843

Files:
  llvm/include/llvm/BinaryFormat/Magic.h
  llvm/include/llvm/Object/Binary.h
  llvm/include/llvm/Object/ObjectFile.h
  llvm/include/llvm/Object/XCOFFObjectFile.h
  llvm/lib/BinaryFormat/Magic.cpp
  llvm/lib/Object/Binary.cpp
  llvm/lib/Object/ObjectFile.cpp
  llvm/lib/Object/SymbolicFile.cpp
  llvm/lib/Object/XCOFFObjectFile.cpp
  llvm/test/tools/llvm-readobj/Inputs/xcoff-basic-64.o
  llvm/test/tools/llvm-readobj/xcoff-basic.test
  llvm/test/tools/llvm-readobj/xcoff-sections.test
  llvm/tools/llvm-readobj/XCOFFDumper.cpp
  llvm/tools/obj2yaml/xcoff2yaml.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63843.208179.patch
Type: text/x-patch
Size: 44735 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190705/2fed1794/attachment.bin>