[PATCH] D149315: [ARM] Don't allocate memory if free space in segmented stack is just enough.

Zhiyao Ma via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 26 18:08:23 PDT 2023


ZhiyaoMa98 created this revision.
ZhiyaoMa98 added reviewers: john.brawn, efriedma.
Herald added subscribers: hiraditya, kristof.beyls.
Herald added a project: All.
ZhiyaoMa98 requested review of this revision.
Herald added a project: LLVM.

Assuming that the stack grows downwards, it is fine if the stack pointer is exactly at the stacklet boundary. We should use less-or-equal condition when deciding whether to skip new memory allocation.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D149315

Files:
  llvm/lib/Target/ARM/ARMFrameLowering.cpp
  llvm/test/CodeGen/ARM/debug-segmented-stacks.ll
  llvm/test/CodeGen/ARM/segmented-stacks-dynamic.ll
  llvm/test/CodeGen/ARM/segmented-stacks.ll
  llvm/test/CodeGen/Thumb/segmented-stacks-dynamic.ll
  llvm/test/CodeGen/Thumb/segmented-stacks.ll
  llvm/test/CodeGen/Thumb2/segmented-stacks.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D149315.517412.patch
Type: text/x-patch
Size: 10915 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230427/10fa65d1/attachment.bin>


More information about the llvm-commits mailing list