[Lldb-commits] [lldb] [lldb] Require paused process and frame for "register info" command (PR #67124)

[via lldb-commits]

jimingham wrote:

process attach --continue

won't return till the process stops when in sync mode.  That's really what sync mode means: "commands that continue the target don't return till the target stops".

But if the process is hitting a breakpoint or exiting and we aren't exiting the `process attach --continue` command in reaction to that, then that is a bug.

Jim


> On Sep 22, 2023, at 7:31 AM, David Spickett ***@***.***> wrote:
> 
> 
> @DavidSpickett commented on this pull request.
> 
> In lldb/test/API/commands/register/register/register_command/TestRegisters.py <https://github.com/llvm/llvm-project/pull/67124#discussion_r1334462296>:
> 
> > +        self.setAsync(True)
> +        self.runCmd("process attach --continue -p %d" % pid)
> What I found was that process attach --continue would never return if you weren't in async mode. Whether that's a bug in itself I'm not sure.
> 
> —
> Reply to this email directly, view it on GitHub <https://github.com/llvm/llvm-project/pull/67124#discussion_r1334462296>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADUPVWYWCVHJNOT672DWAXLX3WOKRANCNFSM6AAAAAA5DBERUM>.
> You are receiving this because your review was requested.
> 



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