[PATCH] D20885: [COFF] Expose the PE debug data directory and dump it

[David Majnemer via llvm-commits]

majnemer added inline comments.

================
Comment at: lib/Object/COFFObjectFile.cpp:462
@@ +461,3 @@
+    uint32_t SectionStart = Section->VirtualAddress;
+    uint32_t SectionEnd = Section->VirtualAddress + Section->VirtualSize;
+    if (SectionStart <= RVA && RVA + Size <= SectionEnd) {
----------------
What about overflow? I reckon we should return an error if SectionEnd < SectionStart.

================
Comment at: tools/llvm-readobj/COFFDumper.cpp:691-692
@@ +690,4 @@
+    } else {
+      // FIXME: Types of 0xC and 0xD are commonly observed but are not in the
+      // documented type enum.  Figure out what they mean.
+      ArrayRef<uint8_t> RawData;
----------------
Really, 12 and 13?


http://reviews.llvm.org/D20885