[llvm] [MachineFrameInfo] Refactoring with computeMaxcallFrameSize() (NFC) (PR #78001)

Diana Picus via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 16 00:55:21 PST 2024


rovka wrote:

> This refactoring looks like an improvement to me, but I have one open question and that is regarding global isel as it would also need to set AdjustsStack..?

You could probably set it in [lowerInlineAsm](https://github.com/llvm/llvm-project/blob/705d9273c5417e04dc542f0e46b90960c235c753/llvm/lib/CodeGen/GlobalISel/InlineAsmLowering.cpp#L215), where the ExtraFlags are computed.

How is this change going to affect code that doesn't pass through isel? For instance if someone writes a MIR test and doesn't specify `adjustsStack` in the `frameInfo`?

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


More information about the llvm-commits mailing list