[Lldb-commits] [lldb] [lldb] Implement basic support for reverse-continue (PR #99736)

[Med Ismail Bennani via lldb-commits]

================
@@ -1881,18 +1970,24 @@ ThreadSP ProcessGDBRemote::SetThreadStopInfo(
           handled = true;
         }
       } else if (!signo) {
-        addr_t pc = thread_sp->GetRegisterContext()->GetPC();
-        lldb::BreakpointSiteSP bp_site_sp =
-            thread_sp->GetProcess()->GetBreakpointSiteList().FindByAddress(pc);
-
-        // If a thread is stopped at a breakpoint site, set that as the stop
-        // reason even if it hasn't executed the breakpoint instruction yet.
-        // We will silently step over the breakpoint when we resume execution
-        // and miss the fact that this thread hit the breakpoint.
-        if (bp_site_sp && bp_site_sp->ValidForThisThread(*thread_sp)) {
-          thread_sp->SetStopInfo(StopInfo::CreateStopReasonWithBreakpointSiteID(
-              *thread_sp, bp_site_sp->GetID()));
+        if (m_last_run_direction == eRunReverse) {
----------------
medismailben wrote:

nit: let's keep the `forward` case the main branch since this should be the default case most of the time.

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