Re: [Lldb-commits] [PATCH] D33831: Add temp_failure_retry helper function

2017-06-02 Thread Zachary Turner via lldb-commits
Maybe just ask on llvm dev what people's thoughts are? On Fri, Jun 2, 2017 at 7:51 AM Pavel Labath via Phabricator < revi...@reviews.llvm.org> wrote: > labath updated this revision to Diff 101212. > labath added a comment. > > - fix freebsd typo > - use ::waitpid consistently > > > https://reviews

Re: [Lldb-commits] [PATCH] D33831: Add temp_failure_retry helper function

2017-06-02 Thread Pavel Labath via lldb-commits
I've wondered about that as well.. I've seen a couple of places where this could be useful, but not nearly as many as in lldb. I'll let you be the judge of that. On 2 June 2017 at 15:43, Zachary Turner wrote: > I wonder if this should go in LLVM? > > On Fri, Jun 2, 2017 at 7:35 AM Pavel Labath vi

Re: [Lldb-commits] [PATCH] D33831: Add temp_failure_retry helper function

2017-06-02 Thread Zachary Turner via lldb-commits
I wonder if this should go in LLVM? On Fri, Jun 2, 2017 at 7:35 AM Pavel Labath via Phabricator < revi...@reviews.llvm.org> wrote: > labath created this revision. > Herald added subscribers: mgorny, emaste. > > It is intended to wrap functions which can fail with EINTR (which we > have a surprisin