probinson added a comment.

So, I've been attacking the type-units-in-.debug_info problem in LLVM, and what 
I've done (so far) in the LLVM parser is to factor out a DWARFUnitHeader class, 
and made DWARFUnit derive from that.  This makes it feasible to parse a unit 
header before deciding what kind of unit we are looking at, which is necessary 
when .debug_info contains both type and compile units.  I haven't posted that 
patch yet as it's an NFC prep for more work getting to the point of actually 
allowing mix-n-match units in .debug_info sections.  But I'll post that patch 
today so people can at least look at it.

Mainly Greg and I should not take diverging approaches when we can converge 
instead.


https://reviews.llvm.org/D32167



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to