Details
- Reviewers
sivachandra asb - Commits
- rG4d94a0080e14: [libc] Support more functions in riscv
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Out of more than 1600 tests (that run in my x86_64 machine), this patch enables 1057 of them.
Also, sorry for the big patch, the next ones shall be smaller/more directed to individual lib support.
Fixed copy & paste error in FMA.h:
LLVM_LIBC_SRC_SUPPORT_FPUTIL_AARCH64_FMA_H should've been LLVM_LIBC_SRC_SUPPORT_FPUTIL_RISCV64_FMA_H.
Hi, thanks for the patch. You have included thread functions but do they build and pass? Their linux implementation is very closely tied to the startup implementation which you have not included in this patch.
Also, can you split this up so that it is easy for me to test? Likewise, send the changes doing cleanups separately.
Two keep it easy to test and rollout, lets also first only add non-fullbuild functions. I have patches ready for this but since you have sent it before me, I will wait for you to send them out. For the syscall implementation in riscv64/syscall.h, lets follow the aarch64/syscall.h pattern.
Hi,
Can you separate this out into following patches:
- Just simple expansion of entrypoints.txt with functions that can now be included without any other changes. Do not include any commented out functions.
- FMA patch.
- sqrt patch.
- Adding riscv fenv_t and singal.h macros. Enable full build also with this patch.
Thanks,
Siva Chandra
I see more functions commented out. Commenting out is fine as long as we include a good reason why it has been commented out.