<div dir="ltr"><br><div class="gmail_extra" style>Some nitpicking:</div><div class="gmail_extra"><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

+  FrameEntry(FrameKind K, DataExtractor D, uint64_t Offset, uint64_t Length)<br>
+    : Kind(K), Data(D), Offset(Offset), Length(Length)<br>
+  {}<br>
+<br></blockquote><div><br></div><div style>Probably want to move the  {} to the previous line. There are a couple of places in this file.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

+  uint64_t LinkedCIEOffset;<br>
+  uint64_t InitialLocation;<br>
+  uint64_t AddressRange;<br>
+  CIE *LinkedCIE;<br>
</blockquote><div><br></div><div style>Some more comments for instance variables.</div><div>  </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+DWARFDebugFrame::DWARFDebugFrame()<br>
+{<br>
+}<br>
+<br></blockquote><div><br></div><div style>Formatting.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+<br>
+DWARFDebugFrame::~DWARFDebugFrame()<br>
+{<br>
+  for (EntryVector::iterator I = Entries.begin(), E = Entries.end();<br>
+       I != E; ++I) {<br>
+    delete *I;<br>
+  }<br>
+}<br>
+<br></blockquote><div><br></div><div style>Ditto.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+<br>
+static void LLVM_ATTRIBUTE_UNUSED dumpDataAux(DataExtractor Data,<br>
+                                              uint32_t Offset, int Length) {<br>
+ <br></blockquote><div><br></div><div style>Double check this formatting?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+    // TODO: For honest DWARF64 support, DataExtractor will have to treat<br>
+    //       offset_ptr as uint64_t*<br>
+    uint32_t EndStructureOffset = Offset + static_cast<uint32_t>(Length);<br>
+<br></blockquote><div><br></div><div style>File a bug for this please?</div><div><br></div><div style>Thanks!</div><div style><br></div><div style>-eric</div></div></div></div>