[PATCH] D156113: [CodeGen] Store call frame size in MachineBasicBlock

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 24 05:43:03 PDT 2023


foad added a comment.

This is a rework of D154281 <https://reviews.llvm.org/D154281>:

- Fixes the ARM bug reported by @olista01. ARM frame index elimination was relying on SPAdj being passed in as 0 if call frame pseudos were eliminated early (the canSimplifyCallFramePseudos case).
- Stores "call frame size" instead of "SP adjustment". This feels a bit more canonical and maps directly to the call frame size stored in call frame setup/destroy pseudos.
- Adds machine verification and a bunch of fixes that that required.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D156113/new/

https://reviews.llvm.org/D156113



More information about the llvm-commits mailing list