[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 created this revision.
luismarques added reviewers: asb, Michael, rogfer01.
Herald added subscribers: vkmr, frasercrmck, evandro, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, edward-jones, zzheng, jrtc27, shiva0217, kito-cheng, niosHD, sabuasal, simoncook, johnrusso, rbar, hiraditya.
luismarques requested review of this revision.
Herald added subscribers: llvm-commits, MaskRay.
Herald added a project: LLVM.

As it stands, this patch is basically just a copy of D97514 <https://reviews.llvm.org/D97514>. Originally I suspected that there might be a better way to solve it, but eventually I realized I had arrived at roughly the same approach. I'm not sure there's much point in using `getRegisterTypeForCallingConv` and `getNumRegistersForCallingConv` versus something more direct, but I've left that unchanged.

It looks like the alloca stack offset is overly aligned?


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D99068

Files:
  llvm/lib/Target/RISCV/RISCVISelLowering.cpp
  llvm/test/CodeGen/RISCV/stack-slot-size.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D99068.332241.patch
Type: text/x-patch
Size: 4162 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210322/458f5aee/attachment.bin>