[PATCH] D69148: Disable exit-on-SIGPIPE in lldb

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 18 10:30:20 PDT 2019


vsk planned changes to this revision.
vsk added a comment.

The death tests are flaky. I've noticed two issues:

1. When run under lit, the DisableExitOnSIGPIPE doesn't actually exit when it receives SIGPIPE. Dtrace suggests that the unit test process inherits an "ignore" sigaction from its parent.
2. When run in parallel, exiting causes the unit test to run atexit destructors for other unit tests lumped into the SupportTests binary. One of these is a condition_variable destructor and it hangs. Also, gtest warns: `Death tests use fork(), which is unsafe particularly in a threaded context. For this test, Google Test detected 21 threads.`

So I plan to give up on testing "EnableExitOnSIGPIPE" and will write a non-death test to get some coverage.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69148/new/

https://reviews.llvm.org/D69148





More information about the llvm-commits mailing list