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

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

Can you describe the cost model in the patch description?



================
Comment at: llvm/lib/Target/RISCV/RISCVFrameLowering.cpp:1495
+      // can be compressed(C.ADDI16SP, offset can be [-512, 496]), but
+      // addi sp, sp, 512 can not be compressed. so try to use 496 firstly.
+      // RVCompressLen - StackAlign = 512 - 16 = 496, it satisfies the
----------------
firstly -> first


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158623