labath added a comment.

It seems Thread plans already have logic 
<https://github.com/llvm/llvm-project/blob/62ec4ac90738a5f2d209ed28c822223e58aaaeb7/lldb/source/Target/ThreadPlanBase.cpp#L133>
 to handle "exec" events. It would be better to figure out why that logic is 
not sufficient instead of adding another exec handling hook.

I am also surprised by the usage of the word "race". It seems to me that 
problem (what exactly is the problem? How do the thread plans "fail" ?) should 
be reproducible deterministically. The plans run when the process is stopped, 
and they should be deterministic. And we can control what the inferior does. 
Can you describe the problem that you're fixing in more detail?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93874/new/

https://reviews.llvm.org/D93874

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to