[llvm-readobj] Include the PE magic value in printouts

[mstorsjo]

Summary

This is useful for a testcase in lld.

[ruiu]

tools/llvm-readobj/COFFDumper.cpp

681

I believe this is 0x8086 for x86, so you want to print it out in hex.

[mstorsjo]

tools/llvm-readobj/COFFDumper.cpp

681

No, it's not that (that's the COFF machine magic word); this is the PE type indicator, 0x10b for normal PE32 and 0x20b for PE32+ (for 64 bit object files). But you're right that it might make more sense to print it in hex.

[mstorsjo]

Printing the value as hex.

[ruiu]

Basically every change needs a test. Can you add?

[mstorsjo]

Added a check for this value in an existing test.

[ruiu]

Can you also add a check to llvm/test/tools/llvm-readobj/file-headers.test?

[mstorsjo]

Added a test for this in the only section of file-headers.test that included ImageOptionalHeader.

[ruiu]

LGTM

tools/llvm-readobj/COFFDumper.cpp

void COFFDumper::printDOSHeader(const dos_header *DH) {
   W.printNumber("OEMid", DH->OEMid);
   W.printNumber("OEMinfo", DH->OEMinfo);
   W.printNumber("AddressOfNewExeHeader", DH->AddressOfNewExeHeader);
 }
 
 template <class PEHeader>
 void COFFDumper::printPEHeader(const PEHeader *Hdr) {
   DictScope D(W, "ImageOptionalHeader");
+  W.printHex   ("Magic", Hdr->Magic);
   W.printNumber("MajorLinkerVersion", Hdr->MajorLinkerVersion);
   W.printNumber("MinorLinkerVersion", Hdr->MinorLinkerVersion);
   W.printNumber("SizeOfCode", Hdr->SizeOfCode);
   W.printNumber("SizeOfInitializedData", Hdr->SizeOfInitializedData);
   W.printNumber("SizeOfUninitializedData", Hdr->SizeOfUninitializedData);
   W.printHex   ("AddressOfEntryPoint", Hdr->AddressOfEntryPoint);
   W.printHex   ("BaseOfCode", Hdr->BaseOfCode);
   printBaseOfDataField(Hdr);