[Lldb-commits] [lldb] [lldb-server] Add type info for qMemoryinfo reponse (PR #209235)

[Jason Molenda via lldb-commits]

================
@@ -2971,6 +2971,13 @@ GDBRemoteCommunicationServerLLGS::Handle_qMemoryRegionInfo(
 
     if (std::optional<unsigned> protection_key = region_info.GetProtectionKey())
       response.Printf("protection-key:%" PRIu32 ";", *protection_key);
+
+    // Type: stack/heap classification, when known.
+    LazyBool is_stack = region_info.IsStackMemory();
+    if (is_stack == eLazyBoolYes)
+      response.PutCString("type:stack;");
+    else if (is_stack == eLazyBoolNo)
+      response.PutCString("type:heap;");
----------------
jasonmolenda wrote:

I don't know linux VM but I don't think "if not stack, is heap" is a good assumption.  We could be looking at shared memory or binary data or other VM types that are not heap allocations.

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