[PATCH] D91719: [RISCV] Add support for using -msave-restore with tailcalls

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 22 03:40:33 PDT 2021


asb added a comment.

In D91719#2843627 <https://reviews.llvm.org/D91719#2843627>, @edward-jones wrote:

> I've rebased and added an option so that the tailcall version of save/restore is only used if `-mllvm -riscv-save-restore-tailcall` is provided.

Thanks Ed, I think making it opt-in like this makes the decision to merge or not slightly easier, but ideally we can get at least some confirmation in https://github.com/riscv/riscv-toolchain-conventions/pull/10 as to whether anybody has major concerns about going in this direction.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D91719/new/

https://reviews.llvm.org/D91719



More information about the llvm-commits mailing list