jasonmolenda added inline comments.

================
Comment at: lldb/source/Target/SectionLoadList.cpp:265-266
        ++pos) {
-    s.Printf("addr = 0x%16.16" PRIx64 ", section = %p: ", pos->first,
-             static_cast<void *>(pos->second.get()));
     pos->second->Dump(s.AsRawOstream(), s.GetIndentLevel(), target, 0);
----------------
mib wrote:
> Out of curiosity, why did you remove the section part ?
I was being grumpy when I saw all the fields SectionLoadList::Dump was 
outputting and removed one of them to make me feel better, but I have to 
acknowledge that seeing the actual Section* value for the section may be 
helpful when debugging lldb.  I'll remove this part of the change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154169

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

Reply via email to