[Lldb-commits] [PATCH] D88387: Create "skinny corefiles" for Mach-O with process save-core / reading

Jason Molenda via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Oct 2 23:08:46 PDT 2020

jasonmolenda marked 3 inline comments as done.
jasonmolenda added inline comments.

Comment at: lldb/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp:6223
+              range_info.GetDirtyPageList();
+          if (dirty_page_list.hasValue()) {
+            for (addr_t dirtypage : dirty_page_list.getValue()) {
clayborg wrote:
> If we add a bool argument, we might need to return an error if the lldb-server doesn't support the dirty page list stuff. Some regions won't have dirty pages, but we might need add detection for any dirty pages and then error out at the end if user requested a minimal core file
in the qMemoryRegionInfo packet response, a remote stub that can identify dirty pages should include a dirty-pages: key-value entry in every response where they're supported, even if it's an empty list.  I clarified this in the docs.  That's how we'll detect the difference between "no dirty pages" and "dirty pages not supported".

  rG LLVM Github Monorepo



More information about the lldb-commits mailing list