[PATCH] D125382: [RISCV] Add a test w/ RVV stack objects misaligning non-RVV ones

[Fraser Cormack via Phabricator via llvm-commits]

frasercrmck added a comment.

I think this can be exhibited even more simply. If we have one RVV object and call a function, that called function expects its stack to be 16-byte aligned. We're not ensuring that.

Of course, most of the time, VLEN is at least 128 so this isn't a problem. I wonder if we can use the architectural knowledge of VLEN to omit any alignment if VLEN >= 128. That doesn't fix our problem with RVV alignments > 16, but at least it brings us up to par with the scalar frame layout.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125382