diff options
author | Ilia K <ki.stfu@gmail.com> | 2015-04-15 13:35:49 +0000 |
---|---|---|
committer | Ilia K <ki.stfu@gmail.com> | 2015-04-15 13:35:49 +0000 |
commit | 725abcb88979402fdb1e3567824a8d1ca2292388 (patch) | |
tree | 21a2fdabdb0b2b970e5f4ea582bb70d031d73a3e /llvm/lib/CodeGen/MachineBlockPlacement.cpp | |
parent | cfb2b9d755584dfc74b1a7e3041b1fa5a10b3209 (diff) | |
download | bcm5719-llvm-725abcb88979402fdb1e3567824a8d1ca2292388.tar.gz bcm5719-llvm-725abcb88979402fdb1e3567824a8d1ca2292388.zip |
Bug 23051 - Fix zombie processes after lldb-server tests
Summary:
This patch fixes the following bug: https://llvm.org/bugs/show_bug.cgi?id=23181
For some reason some lldb-server tests should be kicked using SIGHUP and SIGINT before termination, otherwise it will leave a zombie process.
I think the reason is that the lldb-server will terminate a slave process if it gets the SIGHUP/SIGINT and if so it should be fixed in lldb-server.
The solution is to terminate process like it does the pexpect (including the delayafterterminate interval).
Also this patch reverts the following commits:
* r234549 - Skip lldb-server tests according to bug 23181
* r234765 - Skip a few tests on OS X according to the bug 23181
* r234803 - Skip the TestGdbRemoteRegisterState.test_grp_register_save_restore_works_no_suffix_debugserver_dsym test on OS X according to the bug 23181
Test Plan:
```
$ ps | grep Z | grep a.out | wc -l
447
$ ./dosep.py --options "-v --executable $INSTALLDIR/bin/lldb"
[...]
$ ps | grep Z | grep a.out | wc -l
447
```
Reviewers: jasonmolenda, clayborg, tberghammer
Reviewed By: clayborg
Subscribers: lldb-commits, dawn, tberghammer, clayborg, jasonmolenda
Differential Revision: http://reviews.llvm.org/D9019
llvm-svn: 235008
Diffstat (limited to 'llvm/lib/CodeGen/MachineBlockPlacement.cpp')
0 files changed, 0 insertions, 0 deletions