[Lldb-commits] [lldb] [lldb-dap] Followup fixs for data breakpoints (PR #81680)

[Greg Clayton via lldb-commits]

================
@@ -2757,13 +2769,18 @@ void request_dataBreakpointInfo(const llvm::json::Object &request) {
       body.try_emplace("description", error_cstr && error_cstr[0]
                                           ? std::string(error_cstr)
                                           : "evaluation failed");
-    } else
-      addr = llvm::utohexstr(value.GetValueAsUnsigned());
+    } else {
+      uint64_t value_as_unsigned = value.GetValueAsUnsigned();
+      if (value_as_unsigned == 0) {
+        body.try_emplace("dataId", nullptr);
+        body.try_emplace("description",
+                         "unable to evaluate expression to an address.");
+      }
----------------
clayborg wrote:

Avoid this and do the memory region code I show below to verify the address in somewhere we can access.

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