[PATCH] D155793: [Support] Avoid wait4 on Fuchsia
Roland McGrath via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 20 09:52:51 PDT 2023
mcgrathr added a comment.
What is implementing waitpid there? I'm not aware of any Fuchsia implementation of that API. The function in libc is an ENOSYS stub. If there were an emulation implementation of waitpid, there could as well be one for wait4 since some of the information analogous to `struct rusage` is accessible by other means on Fuchsia. But since there isn't AFAIK any waitpid et al that is actually useful on Fuchsia today, I'm a bit confused by this change. It seems like perhaps it's just getting something to compile and link that doesn't actually work. IMHO unless and until we provide a more full posix_spawn+wait* emulation library for Fuchsia--and perhaps even if we had one--the LLVM support should probably use the more direct Fuchsia mechanisms instead. For spawning subprocesses, that means fdio_spawn and then the direct Zircon syscall APIs to wait for process termination.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D155793/new/
https://reviews.llvm.org/D155793
More information about the llvm-commits
mailing list