[PATCH] D128430: [LoongArch] Support lowering FrameIndex

[Ray Wang via Phabricator via llvm-commits]

wangleiat added inline comments.


================
Comment at: llvm/lib/Target/LoongArch/LoongArchFrameLowering.cpp:68-69
+  int MaxCSFI = -1;
+  StackOffset Offset =
+      StackOffset::getFixed(MFI.getObjectOffset(FI) - getOffsetOfLocalArea() +
+                            MFI.getOffsetAdjustment());
----------------
xen0n wrote:
> This implementation is a simplified version of RISCV's `getFrameIndexReference`, but we're missing a stack ID check before this. I know we're not utilizing any non-default stack IDs so far but I don't know if such a check/assertion should be kept anyway.
Riscv uses stackID to handle `ScalableVector`, which we currently don't support, at least not currently.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128430