[PATCH] D105116: [AMDGPU] Set optional PAL metadata

[Jay Foad via Phabricator via llvm-commits]

foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUAsmPrinter.cpp:1406
+  MD->setFunctionLdsSize(MF, CurrentProgramInfo.LDSSize);
+  MD->setFunctionNumUsedVgprs(MF, CurrentProgramInfo.NumVGPRsForWavesPerEU);
+  MD->setFunctionNumUsedSgprs(MF, CurrentProgramInfo.NumSGPRsForWavesPerEU);
----------------
Is this the number of vgprs that are actually used by the generated ISA? From the name "NumVGPRsForWavesPerEU" it sounds like it might be the maximum number that //could// be used while still achieving some specified occupancy.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105116/new/

https://reviews.llvm.org/D105116