[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