[Lldb-commits] [lldb] [lldb] Read `read-only` pointee data from the object file in `GetPointeeData` (PR #213372)

[Piyush Jaiswal via lldb-commits]

================
@@ -764,7 +764,7 @@ size_t ValueObject::GetPointeeData(DataExtractor &data, uint32_t item_idx,
         target_addr.SetLoadAddress(addr + offset, target);
         size_t bytes_read =
             target->ReadMemory(target_addr, heap_buf_ptr->GetBytes(), bytes,
-                               error, /*force_live_memory=*/true);
+                               error, /*force_live_memory=*/false);
----------------
piyushjaiswal98 wrote:

Thanks @jimingham . Updated the patch to read live process memory first and only fall back to `force_live_memory=false` if the live read fails, so we still prefer live memory while recovering read-only data that isn't backed by the core file. Added unit tests for both paths. 

https://github.com/llvm/llvm-project/pull/213372