[libc-commits] [libc] [libc] End a semaphore wait with EINTR when a signal handler runs (PR #236969)

[Pavel Labath via libc-commits]

https://github.com/labath commented:

The change makes sense functionality-wise. The thing I'm not sure about is the fourth unnamed function argument. It makes it easy to mix them up, and I'm not even sure it's the right default, as libc functions typically do pass EINTRs back to the caller.

I'm considering having the function always return EINTR, and then wrapping the call sites with some TEMP_FAILURE_RETRY-like helpers -- but I'd like to hear what others think too...

https://github.com/llvm/llvm-project/pull/236969