[PATCH] D53000: [Support] exit with custom return code for SIGPIPE

[Jonas Devlieghere via Phabricator via llvm-commits]

JDevlieghere added a comment.

In D53000#1430211 <https://reviews.llvm.org/D53000#1430211>, @jfb wrote:

> In D53000#1430182 <https://reviews.llvm.org/D53000#1430182>, @JDevlieghere wrote:
>
> > Hey Nick,
> >
> > This change is causing problems in LLDB. We want to ignore `SIGPIPE` and this appears to be making that impossible, which doesn't sound fair for something like libSupport. I'm not at all familiar with this code, but can we at least check if the signal is supposed to be ignored here?
>
>
> I think you'll want to set up something like `InterruptFunction`: by default we'll do what Nick added, but if you ask we can do something else for `SIGPIPE`.


Wouldn't that affect all signals? What about an atomic boolean to enable/disable this behavior? Like I said before, I personally don't think this it is expected from a library like Support to do this. It feels like this behavior should be opt-in.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D53000