[Lldb-commits] [lldb] [lldb] Fix race condition in Process::WaitForProcessToStop() (PR #144919)

[via lldb-commits]

athierry-oct wrote:

@labath ok, thanks for explaining why HandleCommand() works this way

So in synchronous mode, should `EvaluateExpression()` (and more specifically, `RunThreadPlan()`) avoid rebroadcasting the stop event when expression evaluation fails due to a process stop? I may be missing something, but it seems to me that rebroadcasting the event is only necessary in async mode, since there’s an actual driver that needs to be notified. In sync mode, to find out what happened, the user can simply read the error from the function they used to evaluate the expression, e.g., `SBFrame::EvaluateExpression()`

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