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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
