[PATCH] D135757: [LoongArch] Add emergency spill slot for GPR for large frames

wanglei via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 12 17:53:47 PDT 2022


wangleiat added inline comments.


================
Comment at: llvm/lib/Target/LoongArch/LoongArchFrameLowering.cpp:168
+  unsigned Addi = STI.is64Bit() ? LoongArch::ADDI_D : LoongArch::ADDI_W;
+  if (hasFP(MF) || StackSize > 49152) {
+    // 49152 = 24*2048, 24 adjustments are enough to meet the conditions of
----------------
Wrong logic, I will modify it to enable `fp` when the size is greater than 4096 and frame-pointer=none is not specified.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135757



More information about the llvm-commits mailing list