[PATCH] D99068: [RISCV][WIP][RFC] Fix stack slot for argument type sizes not a multiple of 64 bits (Bug 49500)

[Luís Marques via Phabricator via llvm-commits]

luismarques added a comment.

In D99068#2641075 <https://reviews.llvm.org/D99068#2641075>, @frasercrmck wrote:

> My fix was more direct: I went over `ArgValue` and all of the `SDValue PartValue = OutVals[i + 1];` as in the loop later in this block, and summed up all of the required `getStoreSize`s of their VTs, and took the maximum required alignment of each. Then created a stack temporary of that summed size and max alignment. I wasn't sufficiently happy with it and since I didn't get much indication that this was an urgent fix I didn't put it up for discussion - sorry about that. I can share my patch with you if you're interested.

That sounds like it has the potential for being a better alternative than this one. Please post it for review. Thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99068