[Lldb-commits] [lldb] Fix a bug with cancelling "attach -w" after you have run a process previously (PR #65822)

Greg Clayton via lldb-commits lldb-commits at lists.llvm.org
Sat Sep 9 11:57:19 PDT 2023


https://github.com/clayborg commented:

A different way to approach this fix is to just fix ExecutionContext so that this can never be out of date by removing the "m_process_sp" member variable. Is is not great that we can an execution context that has an older process, and would we ever want an ExecutionContext to be able to have a mismatched process? If we remove m_process_sp, we can always get the process from the m_target_sp and then we can never get it wrong that way and we might avoid some of the changes in this patch? Lemme know what you think as It would be good to make sure our ExecutionContext variables can't get the wrong info.

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


More information about the lldb-commits mailing list