[libc-commits] [PATCH] D76267: [libc] Resolve race condition in sub-process test runner.

[Alex Brachet via Phabricator via libc-commits]

abrachet accepted this revision.
abrachet added a comment.

LGTM. Sorry about that, I never caught this failing so thanks for catching it!

> however if the child closes the pipe intentionally then it could result in the test runner hanging. I find this situation less likely, where as I experience failures locally with this race condition rather consistently.

Agreed I don't think the death test would run `::close(4)`, and this is a more controlled environment so I think it is fine. If we want we can do the more traditional pattern of setting an alarm and sigwaiting for either `SIGALRM` or `SIGCHLD`, but I find signals very hacky.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76267