wangpc-pp wrote: > Side note: shouldn't we also update `compiler-rt/lib/builtins/riscv/{save, > restore}.S`? E.g. with something like this: > [...] > (I don't remember why exactly since I did it a long time ago, but for some > reason I do have this patch in my LLVM fork, so it probably was necessary for > _something_.)
Oh, it may be about stack alignment issue for libcalls. I may have fixed it. As for your diffs, it seems that you only handle the `__riscv_save/restore_[2|1|0]`, which is incomplete. And the code is not different with non-rve cases? https://github.com/llvm/llvm-project/pull/76777 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits