Check the breakpoint has resolved locations

Author: delcypher

lldb/source/Plugins/InstrumentationRuntime/BoundsSafety/InstrumentationRuntimeBoundsSafety.cpp

@@ -314,8 +314,15 @@ void InstrumentationRuntimeBoundsSafety::Activate() {
       /*internal=*/true,
       /*request_hardware*/ false);
 
+  // TODO: Once we have an instrumentation log handle we should log these
+  // failures (rdar://164920875).
   if (!breakpoint)
     return;
+  if (!breakpoint->HasResolvedLocations()) {
+    assert(0 && "breakpoint has no resolved locations");
+    process_sp->GetTarget().RemoveBreakpointByID(breakpoint->GetID());
+    return;
+  }
 
   // Note: When `sync=true` the suggested stackframe is completely ignored. So
   // we use `sync=false`. Is that a bug?