[PATCH] D158150: [AMDGPU] Add dynamic LDS size implicit argument to v5

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUHSAMetadataStreamer.cpp:1106
+  // emit argument for hidden dynamic lds size
+  emitKernelArg(DL, Int64Ty, Align(8), "hidden_dynamic_lds_size", Offset, Args);
 }
----------------
arsenm wrote:
> This just just be an i32
It also doesn't need 8 byte alignment


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158150