[PATCH] D99087: [RISCV][WIP] Fix stack slot for argument types (Bug 49500)
Alex Bradbury via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 1 07:10:48 PDT 2021
asb added a comment.
@frasercrmck you mentioned this patch doesn't quite seem right to you - is there a particular part you're concerned about. I've spent some time going over both this and D99068 <https://reviews.llvm.org/D99068>. The logic of this patch makes more intuitive sense to me, but that may be as it's a long time since I have a better memory of the logic it's modifying. Though Luis' approach does indeed seem to reduce stack size in some cases, and of course directly mirrors the matching SystemZ fix.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D99087/new/
https://reviews.llvm.org/D99087
More information about the llvm-commits
mailing list