[PATCH] D123878: [AMDGPU] Add remarks to output some resource usage

[Vang Thao via Phabricator via llvm-commits]

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