[PATCH] D158623: [RISCV] Reorder the stack frame objects.

Wang Pengcheng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 25 00:37:06 PDT 2023


wangpc added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVFrameLowering.cpp:566
+  // It's only used to reduce codesize.
+  if (!MF.getFunction().hasMinSize())
+    return;
----------------
It seems we can also reduce stack size? I think we can enable it by default, not only for optsize. Performance should be evaluated.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158623



More information about the llvm-commits mailing list