<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Mon, Aug 5, 2013 at 11:04 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  Few questions inline, otherwise looks pretty good.<br>
<br>
<br>
================<br>
Comment at: include/llvm/DebugInfo/DIContext.h:127<br>
@@ +126,3 @@<br>
+  enum DIContextKind {<br>
+    DICK_DWARF<br>
+  };<br>
----------------<br>
Unfortunate naming in english. How about CK_DWARF?<br>
<br>
================<br>
Comment at: lib/DebugInfo/DWARFContext.h:30<br>
@@ -29,3 +29,3 @@<br>
 class DWARFContext : public DIContext {<br>
-  SmallVector<DWARFCompileUnit, 1> CUs;<br>
+  SmallVector<DWARFCompileUnit *, 1> CUs;<br>
   OwningPtr<DWARFDebugAbbrev> Abbrev;<br>
----------------<br>
Consider breaking this part out first in a separate patch? Also, since there's a single DWO for a single compile unit I'm not sure about the change to pointer as far as relevance to this patch. Can you give a bit more explanation here?<br>
</blockquote><div><br></div><div>Sure, I can submit bits of this patch separately (RTTI for DIContext and this change). For simplicity, I make OwningPtr<DWARFCompileUnit> DWOCU a member of DWARFCompileUnit class (so that each compile unit may store and own .dwo associated with it). But then DWARFCompileUnit is not trivially copyable anymore, so we can't store it (or OwningPtr<DWARFCompileUnit>) in vector anymore.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D1164" target="_blank">http://llvm-reviews.chandlerc.com/D1164</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Alexey Samsonov, MSK</div>
</div></div>