jingham added a comment.

That test is good.  That tests the lazy lookup of the dwarf type indices.  The 
other thing this changes is symbol reading.  Can you also add a similar test 
that relies on finding a symbol in a shared library we might not have read in?  
It probably fine to add it to this test case...  It's unlikely we'll break lazy 
symbol reading and not also break lazy Dwarf Index building, but you never 
know.  Other than that, this looks fine to me.


Repository:
  rL LLVM

https://reviews.llvm.org/D32598



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

Reply via email to