[Lldb-commits] [lldb] [lldb] Use `Address` to setup breakpoint (PR #94794)

Jonas Devlieghere via lldb-commits lldb-commits at lists.llvm.org
Fri Jun 28 07:50:39 PDT 2024


================
@@ -90,17 +90,9 @@ void InstrumentationRuntimeASanLibsanitizers::Activate() {
   if (!process_sp)
     return;
 
-  lldb::ModuleSP module_sp = GetRuntimeModuleSP();
-
   Breakpoint *breakpoint = ReportRetriever::SetupBreakpoint(
-      module_sp, process_sp, ConstString("sanitizers_address_on_report"));
-
-  if (!breakpoint) {
-    breakpoint = ReportRetriever::SetupBreakpoint(
-        module_sp, process_sp,
-        ConstString("_Z22raise_sanitizers_error23sanitizer_error_context"));
-  }
-
+      GetRuntimeModuleSP(), process_sp,
+      ConstString("sanitizers_address_on_report"));
----------------
JDevlieghere wrote:

@clayborg The old code had a fallback to `_Z22raise_sanitizers_error23sanitizer_error_context` but this breakpoint is still the right one. 

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


More information about the lldb-commits mailing list