labath accepted this revision.
labath added a comment.
This revision is now accepted and ready to land.

Thanks, this looks much better.

In D61498#1490885 <https://reviews.llvm.org/D61498#1490885>, @jankratochvil 
wrote:

> OK, I agree; originally I did not want to touch more code than what I needed 
> for .debug_types.


I think that "not wanting to touch more code than needed" was the reason why 
all previous attemtps at debug_types were hacky in one way or another. :)



================
Comment at: lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugInfo.cpp:129-130
+
+  // llvm::lower_bound is not used as for DIE offsets it would still return
+  // index +1 and GetOffset() returning index itself would be a special case.
+  auto pos = llvm::upper_bound(m_compile_units, offset,
----------------
Ah, I think I finally understand the `--pos` thingy. :)


Repository:
  rLLDB LLDB

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61498/new/

https://reviews.llvm.org/D61498



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

Reply via email to