[PATCH] D123878: [AMDGPU] Add remarks to output some resource usage
Vang Thao via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jul 14 10:15:37 PDT 2022
vangthao added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUAsmPrinter.cpp:1222
+ // printing multiple diagnostic location and diag opts.
+ EmitResourceUsageRemark("KernelName", "Kernel Name",
+ MF.getFunction().getName());
----------------
arsenm wrote:
> Why is this kernel name? Do we not emit these for other functions?
We do emit these for other functions. Would this be better off as "Function Name" instead of kernel?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D123878/new/
https://reviews.llvm.org/D123878
More information about the cfe-commits
mailing list