diff options
author | Vedant Kumar <vsk@apple.com> | 2021-01-08 11:08:52 -0800 |
---|---|---|
committer | Vedant Kumar <vsk@apple.com> | 2021-01-08 15:13:04 -0800 |
commit | e05baf40de8a3bbfcf4a765761b1147e94b7309c (patch) | |
tree | ee78b49ebaad0c1c1add313860d31dd4ee3bef5d /clang/lib/Frontend/CompilerInvocation.cpp | |
parent | d86e16e4bd5ed90f08e6e787785b485636a2184d (diff) | |
download | llvm-e05baf40de8a3bbfcf4a765761b1147e94b7309c.zip llvm-e05baf40de8a3bbfcf4a765761b1147e94b7309c.tar.gz llvm-e05baf40de8a3bbfcf4a765761b1147e94b7309c.tar.bz2 |
[InitLLVM] Ensure SIGPIPE handler installed before sigaction()
The pipe signal handler must be installed before any other handlers are
registered. This is because the Unix RegisterHandlers function does not
perform a sigaction() for SIGPIPE unless a one-shot handler is present,
to allow long-lived processes (like lldb) to fully opt-out of llvm's
SIGPIPE handling and ignore the signal safely.
Fixes a bug introduced in D70277.
Tested by running Nick's test case:
% xcrun ./bin/clang -E -fno-integrated-cc1 x.c | tee foo.txt | head
I verified that child cc1 process exits with IO_ERR, and that the parent
recognizes the error code, exiting cleanly.
Differential Revision: https://reviews.llvm.org/D94324
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions