[PATCH] D135895: [RISCV] Limit alignment of stack temporary used for indirect passing of scalable vectors.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 14 08:12:08 PDT 2022


craig.topper added a comment.

In D135895#3857774 <https://reviews.llvm.org/D135895#3857774>, @frasercrmck wrote:

> Does anything in the callee expect/rely on the alignment? I'm wondering if it's possible that a caller with one subtarget uses a lower alignment than its callee with another subtarget, causing issues.

Good catch. I think the load created in LowerFormalArguments uses the ABI align.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135895



More information about the llvm-commits mailing list