[PATCH] D62190: [RISCV] Allow shrink wrapping for RISC-V
Alex Bradbury via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 21 07:39:54 PST 2019
asb added a comment.
I note that the TargetFrameLowering hooks `canUseAsPrologue` and `canUseAsEpilogue` are both called by the shrink wrapper. They default to true, but targets may need to override this for correctness. Looking at e.g. AArch64, I see it overrides `canUseAsPrologue` and returns false in the case that the function needs stack realignment and there are no scratch registers available. Are you certain that no such case is needed for RISC-V?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62190/new/
https://reviews.llvm.org/D62190
More information about the llvm-commits
mailing list