<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Mar 14, 2019 at 11:48 AM Greg Clayton via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">clayborg accepted this revision.<br>
clayborg added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
As long as there is not a large performance regress when parsing large DWARF files this looks good to me. Abbreviation declarations are small, so I don't expect one.<br>
<br>
<br>
<br>
================<br>
Comment at: lldb/source/Plugins/SymbolFile/DWARF/DWARFAbbreviationDeclaration.cpp:37-42<br>
+  if (m_tag == DW_TAG_null) {<br>
+    // FIXME: According to the DWARF spec this may actually be malformed.<br>
+    // Should this return an error instead?<br>
+    return DWARFEnumState::Complete;<br>
+  }<br>
+<br>
----------------<br>
Seems like this would be malformed and should be an error. Something about NULL tag when valid tag was expected<br></blockquote><div><br></div><div>Forgot to address this before submitting.  Will do so in a followup. </div></div></div>