https://github.com/labath commented:

I find it confusing to use the same entry type for 
`Target::SetExecutableModule` and `Process::SetExitStatus`. I think it just 
doesn't match the lldb object model. I suspect you somehow want to bracket the 
existence of a process between a "start" and "exit" events, but there's no 
guarantee that this will do that.

SetExecutableModule can be called both before a process is started and after 
that (multiple times, due to execve). And if you run the same target repeatedly 
you may get multiple SetExitStatus calls without ever changing the executable 
module.

I think we need to slow down and figure out what exactly you want to collect, 
and then find a place to do that.

https://github.com/llvm/llvm-project/pull/127834
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to