[PATCH] D119027: [AMDGPU][NFC]: Emit metadata for hidden_heap_v1 kernarg
Sameer Sahasrabuddhe via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 21 22:13:44 PST 2022
sameerds added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUAttributor.cpp:413
- if (funcRetrievesHostcallPtr(A)) {
- removeAssumedBits(IMPLICIT_ARG_PTR);
removeAssumedBits(HOSTCALL_PTR);
----------------
cfang wrote:
> sameerds wrote:
> > Why was this removed? Is there some other place which guarantees the same relationship?
> I think the explicit call of amdgcn_implicitarg_ptr will guarantee IMPLICIT_ARG_PTR.
That sounds correct. But then this should be asserted somewhere ... probably in the code that manifests the attribute.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119027/new/
https://reviews.llvm.org/D119027
More information about the llvm-commits
mailing list