[PATCH] D21503: [llvm-dwarfdump] - Teach dwarfdump to dump gdb-index section.

[Davide Italiano via llvm-commits]

davide added inline comments.

================
Comment at: include/llvm/DebugInfo/DIContext.h:130
@@ -129,2 +129,3 @@
+  DIDT_GdbIndex,
 };
 
----------------
Unsorted?

================
Comment at: include/llvm/DebugInfo/DWARF/DWARFContext.h:46
@@ -44,2 +45,3 @@
   std::unique_ptr<DWARFUnitIndex> TUIndex;
+  std::unique_ptr<DWARFGdbIndex> GdbIndex;
   std::unique_ptr<DWARFDebugAbbrev> Abbrev;
----------------
Ditto.

================
Comment at: include/llvm/DebugInfo/DWARF/DWARFContext.h:156
@@ -153,1 +155,3 @@
 
+  const DWARFGdbIndex &getGdbIndex();
+
----------------
You can probably group with the other get*Index()

================
Comment at: lib/DebugInfo/DWARF/DWARFGdbIndex.cpp:12
@@ +11,3 @@
+#include "llvm/Support/Format.h"
+#include "llvm/Support/raw_ostream.h"
+
----------------
This include is dead (I think).

================
Comment at: lib/DebugInfo/DWARF/DWARFGdbIndex.cpp:22
@@ +21,3 @@
+               CuList.size())
+     << '\n';
+  uint32_t I = 0;
----------------
I may be wrong, but this seems formatted weirdly. Did you clang-format the patch?

================
Comment at: lib/DebugInfo/DWARF/DWARFGdbIndex.cpp:137
@@ +136,3 @@
+
+  // The symbol table.This is an open - addressed hash table.The size of the
+  // hash table is always a power of 2.
----------------
Space after dot. Also, open addressed (without the --) maybe?


http://reviews.llvm.org/D21503