[PATCH] D112488: AMDGPU: Assume all amdhsa kernarg passed implicit arguments by default

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUSubtarget.cpp:659
+
+  // Assume all implicit inputs are used by default
+  return AMDGPU::getIntegerAttribute(F, "amdgpu-implicitarg-num-bytes", 56);
----------------
arsenm wrote:
> rampitec wrote:
> > It is not immediately obvious we have a HSA kernel at this point.
> This is only meaningfully called with a kernel
With MESA ruled out does that mean only a HSA kernel remains? Maybe add at least an assert?


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

https://reviews.llvm.org/D112488