[Lldb-commits] [PATCH] D58257: Disable ExecControl/StopHook/stop-hook-threads.test on Linux

Jim Ingham via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Feb 14 16:12:49 PST 2019


jingham added a comment.

I haven't gotten a chance to revisit the stop hooks in a long time.  They definitely need some love.

IMO, stop hooks should have the same semantics as breakpoint commands, the first command that causes the target to run should exit the stop hook.  That way you would never get into a situation where you hit another stop hook while handing previous stop's hit.

And we should also add a --continue (maybe also --step & --next) option to the stop hook.  Then we could just disallow step/next/continue in stop hooks.  It is really hard to reason about what to do when handling stops if the contents of the stop hook can restart the target out from under you.  For instance, if you have multiple stop hooks, you have to abort running all the other stop hooks after the first one continues.


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

https://reviews.llvm.org/D58257





More information about the lldb-commits mailing list