DavidSpickett wrote: > The only concern I have is whether it would be acceptable to change the > HostThreadPosix interface directly in this patch? Although it would spare me > the need to manage a separate PR, the community generally prefers small, > separate patches. Nevertheless, if you think it would be appropriate, I could > do it as part of this patch.
Changing the behaviour of Reset would effect use cases other than the monitor thread, and though I'm 99.9% sure it'd be correct to do, doing that change in its own PR would allow us to test that in isolation. Then you can use that new Reset method in this PR to solve this specific issue. https://github.com/llvm/llvm-project/pull/177572 _______________________________________________ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
