[Lldb-commits] [lldb] [lldb][AIX] get host info for AIX (cont..) (PR #138687)
Hemang Gadhavi via lldb-commits
lldb-commits at lists.llvm.org
Fri May 23 05:06:06 PDT 2025
HemangGadhavi wrote:
> > I don't know if this is the source of the problem, but one thing I noticed is that you're missing some sort of synchronization to make sure the child process doesn't exit before you're able to observe it. A slightly lame but probably sufficient method to do that would be to put a long (say, 1 minute, it's there just to ensure the process exits if something goes wrong) sleep into the child, and then have the parent kill it when it is done with it.
>
> The code looks good, but you still need to do something to ensure there's no race in retrieving the process info.
Hi @labath Added some delay which gives some time before retrieving the process info.
Please review once.
https://github.com/llvm/llvm-project/pull/138687
More information about the lldb-commits
mailing list