[PATCH] D136222: [LoongArch] Split SP adjustment

Lu Weining via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 23 06:06:50 PDT 2022


SixWeining accepted this revision.
SixWeining added a comment.
This revision is now accepted and ready to land.

LGTM.



================
Comment at: llvm/lib/Target/LoongArch/LoongArchFrameLowering.cpp:303
+  const std::vector<CalleeSavedInfo> &CSI = MFI.getCalleeSavedInfo();
+  uint64_t StackSize = MFI.getStackSize();
+
----------------
Nit: Inline one-time-used variable.


================
Comment at: llvm/test/CodeGen/LoongArch/stack-realignment.ll:557
 ;
 ; LA64-LABEL: caller4096:
 ; LA64:       # %bb.0:
----------------
With the addition of this patch, 7 instructions are reduced for this function.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136222



More information about the llvm-commits mailing list