[PATCH] D140555: [lld][COFF]llvm-readobj COFFDumper print PEHeader CheckSum

Alexandre Ganea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 26 09:32:00 PST 2022


aganea added inline comments.


================
Comment at: llvm/tools/llvm-readobj/COFFDumper.cpp:740
   W.printNumber("SizeOfHeaders", Hdr->SizeOfHeaders);
+  W.printNumber("CheckSum", Hdr->CheckSum);
   W.printEnum  ("Subsystem", Hdr->Subsystem, makeArrayRef(PEWindowsSubsystem));
----------------
I would call `W.printHex(...)` here instead.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140555



More information about the llvm-commits mailing list