[llvm] [RISCV] Fix stack size computation when M extension disabled (PR #78602)

Simeon K via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 18 09:46:48 PST 2024


simeonkr wrote:

@asb I surmise there may be certain embedded RV32I cores out there that might require it, but we don't have any on our end. However, the code in question was the only place I found where we emit a mul instruction, so I don't anticipate much effort beyond this patch to support this configuration.

The original motivation for this came from the fact that certain tests are defined without `-mattr=+m` and therefore crash when a patch does something to change the stack size to an inconvenient value. I didn't think it would be justified for me to sneak in the attribute simply to hide the crash, hence this patch.

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


More information about the llvm-commits mailing list