[llvm] [RISCV] Take known minimum vlen into account when calculating alignment padding in assignRVVStackObjectOffsets. (PR #110312)

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 30 10:29:28 PDT 2024


topperc wrote:

> I think this is correct, but the (existing) pattern of using fixed offsets to represent a combined scalable + fixed term here makes me uncomfortable that I'm understanding it correctly. I think this code would be much easier to follow if we used TypeSize to the extend we can get away with it through this code.

I'm not sure what you mean by "combined"? TypeSize can only be fixed or scalable. Not both.

https://github.com/llvm/llvm-project/pull/110312


More information about the llvm-commits mailing list