[PATCH] D70277: [Signal] Allow llvm clients to opt into one-shot SIGPIPE handling

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 7 17:53:20 PST 2021


vsk added a comment.

Unfortunately I can’t get to this right now, but will take a look first thing tomorrow morning.

A shot in the dark: looks like your workflow causes a clang process to be sent SIGPIPE more than once. The second time, it exits with ioerr. I expected this to be the pre-patch behavior, so I’m confused as to why the reverts appear to help.

If you need a quick fix, I’d suggest adding a SIG_IGN for SIGPIPE to the clang driver. Those reverts could regress lldb and I’m really not sure why they work.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70277



More information about the llvm-commits mailing list