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

Reply via email to