[PATCH] D141700: AMDGPU: Move enqueued block handling into clang
Matt Arsenault via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 4 08:32:16 PDT 2023
arsenm added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUHSAMetadataStreamer.cpp:299
+
+ Attrs.mRuntimeHandle = getEnqueuedBlockSymbolName(TM, Func);
}
----------------
kzhuravl wrote:
> Do we really need/want to update code object v2?
as long as the code is here yes. Not updating it would mean maintaining two paths in the implementation. This is just changing the internal representation
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D141700/new/
https://reviews.llvm.org/D141700
More information about the cfe-commits
mailing list