[PATCH] D90036: [AMDGPU] Emit stack frame size in metadata

[Jay Foad via Phabricator via llvm-commits]

foad added inline comments.


================
Comment at: llvm/test/CodeGen/AMDGPU/amdpal-callable.ll:104
+; GCN-NEXT:      dynamic_stack:
+; GCN-NEXT:        .stack_frame_size_in_bytes: 0x10
+; GCN-NEXT:      no_stack:
----------------
Flakebi wrote:
> arsenm wrote:
> > This doesn't really represent the unknown nature of the stack size, but I guess there isn't a key for this yet?
> I guess it should be the maximum that the function needs at any point, so 0x10 sounds right?
Then what would it report if there was an alloca in a loop?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90036