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

[Rui Ueyama]

================
Comment at: lib/Object/COFFObjectFile.cpp:467
@@ +466,3 @@
+      PE32Header = 0;
+      return;
+    }
----------------
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.


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