[Lldb-commits] [lldb] r203232 - If you are disabling a hardware breakpoint, use z1 not z0.

Jim Ingham jingham at apple.com
Fri Mar 7 03:18:02 PST 2014


Author: jingham
Date: Fri Mar  7 05:18:02 2014
New Revision: 203232

URL: http://llvm.org/viewvc/llvm-project?rev=203232&view=rev
Log:
If you are disabling a hardware breakpoint, use z1 not z0.

<rdar://problem/16256532>

Modified:
    lldb/trunk/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp

Modified: lldb/trunk/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp?rev=203232&r1=203231&r2=203232&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp (original)
+++ lldb/trunk/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp Fri Mar  7 05:18:02 2014
@@ -2440,8 +2440,16 @@ ProcessGDBRemote::DisableBreakpointSite
             break;
 
         case BreakpointSite::eExternal:
-            if (m_gdb_comm.SendGDBStoppointTypePacket(eBreakpointSoftware, false, addr, bp_op_size))
+            {
+                GDBStoppointType stoppoint_type;
+                if (bp_site->IsHardware())
+                    stoppoint_type = eBreakpointHardware;
+                else
+                    stoppoint_type = eBreakpointSoftware;
+                
+                if (m_gdb_comm.SendGDBStoppointTypePacket(stoppoint_type, false, addr, bp_op_size))
                 error.SetErrorToGenericError();
+            }
             break;
         }
         if (error.Success())





More information about the lldb-commits mailing list