In this patch we add support for the spawn lib in riscv.
Only small changes were required, the biggest one was to use of dup3
instead of dup2, if the latter is not available. This follows our
implementation of dup2.
Paths
| Differential D146145
[libc] Enable spawn lib in riscv ClosedPublic Authored by mikhail.ramalho on Mar 15 2023, 8:42 AM.
Details
Summary In this patch we add support for the spawn lib in riscv. Only small changes were required, the biggest one was to use of dup3
Diff Detail
Event TimelineHerald added projects: Restricted Project, Restricted Project. · View Herald TranscriptMar 15 2023, 8:42 AM mikhail.ramalho added a parent revision: D146065: [libc] Update supported riscv libs.Mar 15 2023, 8:43 AM Comment Actions LGTM with one comment.
Comment Actions We have libc RISCV64 builders now: https://lab.llvm.org/buildbot/#/workers/202 This revision is now accepted and ready to land.Mar 15 2023, 9:45 AM mikhail.ramalho added a child revision: D146161: [libc] Enable __llvm_libc_syscall and fork on riscv64..Mar 15 2023, 12:02 PM Closed by commit rGfe99de31d949: [libc] Enable spawn lib in riscv (authored by Mikhail R. Gadelha <mikhail@igalia.com>). · Explain WhyMar 16 2023, 2:18 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 505730 libc/config/linux/riscv64/entrypoints.txt
libc/config/linux/riscv64/headers.txt
libc/src/spawn/linux/posix_spawn.cpp
|
You should include <signal.h> not the macro file unconditionally. So, no ifdef, just #include <signal.h>.