[PATCH] readobj: Dump PE/COFF optional records.

[Rui Ueyama]

================
Comment at: lib/Object/COFFObjectFile.cpp:467
@@ +466,3 @@
+      PE32Header = 0;
+      return;
+    }
----------------
Rui Ueyama wrote:
> Rafael Ávila de Espíndola wrote:
> > You are not setting an error and returning early, this will leave the file in a inconsistent state (SectionTable is not set for example).
> > 
> > When is the pe header not present? It would be much nicer if we could have just one field named Header of type pe32_header.
> > 
> PE header is present in executable and DLL, and is not in object file. There are two types of PE header, PE32 and PE32+ (64 bit version of PE). The size of PE32+ is different from PE32 and is not handled in this patch.
> 
> My intention was to reset PE32Header if it's PE32+ and fall through to the following code, so that the COFF part of the PE32+ file will be output. Writing return at this line was mistake.
So, because PE header is not present for COFF, we can't have one field named pe32_header. We could remove coff_file_header from pe32_header if we want to avoid the duplication, though.


http://llvm-reviews.chandlerc.com/D939