================
@@ -454,8 +525,13 @@ bool 
BreakpointLocation::SetBreakpointSite(BreakpointSiteSP &bp_site_sp) {
 }
 
 llvm::Error BreakpointLocation::ClearBreakpointSite() {
-  if (!m_bp_site_sp)
-    return llvm::createStringError("no breakpoint site to clear");
+  if (!m_bp_site_sp) {
+    // This might be a Facade Location, which don't have sites or addresses
+    if (IsFacade())
+      return llvm::Error::success();
+    else
+      return llvm::createStringError("no breakpoint site to clear");
----------------
JDevlieghere wrote:

[No else after 
return](https://llvm.org/docs/CodingStandards.html#don-t-use-else-after-a-return).

https://github.com/llvm/llvm-project/pull/158128
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to