[PATCH] D113234: Extend obj2yaml to optionally preserve raw __LINKEDIT/__DATA segments.

[Alexander Shaposhnikov via Phabricator via llvm-commits]

alexander-shaposhnikov added inline comments.


================
Comment at: llvm/lib/Object/MachOObjectFile.cpp:2052
+template <typename LoadCommandType>
+ArrayRef<uint8_t> getSegmentContents(const MachOObjectFile &Obj,
+                                     MachOObjectFile::LoadCommandInfo LoadCmd,
----------------
static ?


================
Comment at: llvm/tools/obj2yaml/macho2yaml.cpp:656
+Error macho2yaml(raw_ostream &Out, const object::Binary &Binary,
+                 bool RawDataSegment, bool RawLinkEditSegment) {
   if (const auto *MachOObj = dyn_cast<object::MachOUniversalBinary>(&Binary))
----------------
in the future someone might want to dump other segments too, though I'm not sure what would be the best solution here 


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

https://reviews.llvm.org/D113234