[PATCH] D128525: Test and document stackmap corner case: when the fram size isn't statically known

Edd Barrett via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 24 06:48:06 PDT 2022


vext01 created this revision.
vext01 added reviewers: arsenm, t.p.northover.
Herald added subscribers: jsji, pengfei.
Herald added a project: All.
vext01 requested review of this revision.
Herald added subscribers: llvm-commits, wdng.
Herald added a project: LLVM.

When a function does a dynamic stack allocation, is it impossible to know the size of the stack statically. If such a function uses stackmaps, then the stack size field is populated with `UINT64_MAX` to indicate this, however this is not tested or documented.

This change tests and documents this case.

Brought to my attention by @ptersilie , thanks.


https://reviews.llvm.org/D128525

Files:
  llvm/docs/StackMaps.rst
  llvm/test/CodeGen/AArch64/stackmap-dynamic-alloca.ll
  llvm/test/CodeGen/SystemZ/stackmap-dynamic-alloca.ll
  llvm/test/CodeGen/X86/stackmap-dynamic-alloca.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D128525.439728.patch
Type: text/x-patch
Size: 3381 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220624/5f62d222/attachment.bin>


More information about the llvm-commits mailing list