[Lldb-commits] [PATCH] D81810: LLDB step-instruction gets stuck on jump to self

2020-06-14 Thread Sebastian Ă–sterlund via Phabricator via lldb-commits
sirmc created this revision. sirmc added a reviewer: LLDB. sirmc added a project: LLDB. Herald added a subscriber: lldb-commits. sirmc edited the summary of this revision. When executing `thread step-inst `/ `si` on an instruction that returns to the same PC, LLDB currently keeps stepping indefin

[Lldb-commits] [PATCH] D81810: LLDB step-instruction gets stuck on jump to self

2020-06-15 Thread Jim Ingham via Phabricator via lldb-commits
jingham added a comment. Just to be clear, the lldb -> gdb command map doesn't prescribe behavior for lldb commands. It just suggests the analogous command in gdb. We are still free to implement lldb behavior however seems best to us. The logic in lldb doesn't change the logic of the program'

[Lldb-commits] [PATCH] D81810: LLDB step-instruction gets stuck on jump to self

2020-06-16 Thread Sebastian Ă–sterlund via Phabricator via lldb-commits
sirmc added a comment. Yes, I wasn't sure what the exact semantics were for step-inst for LLDB. I think the issue is not mainly the mimicking of GDB behavior, but rather that it is inconvenient for some use-cases. To give some context on why I propose this change: the current behavior is prett

[Lldb-commits] [PATCH] D81810: LLDB step-instruction gets stuck on jump to self

2020-06-16 Thread Jim Ingham via Phabricator via lldb-commits
jingham added a comment. In D81810#2095902 , @sirmc wrote: > Yes, I wasn't sure what the exact semantics were for step-inst for LLDB. I > think the issue is not mainly the mimicking of GDB behavior, but rather that > it is inconvenient for some use-cases

[Lldb-commits] [PATCH] D81810: LLDB step-instruction gets stuck on jump to self

2021-03-25 Thread Raphael Isemann via Phabricator via lldb-commits
teemperor requested changes to this revision. teemperor added a comment. This revision now requires changes to proceed. Herald added subscribers: JDevlieghere, pengfei. (Marking as requested-changes to get this out of my review queue) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACT