[PATCH] D28760: [AMDGPU] Add target information that is required by tools to metadata

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/GCNSchedStrategy.cpp:147-148
   unsigned MaxWaves = getMaxWaves(SGPRPressure, VGPRPressure, DAG->MF);
-  unsigned SGPRCriticalLimit = SRI->getMaxNumSGPRs(ST, MaxWaves, true);
-  unsigned VGPRCriticalLimit = SRI->getMaxNumVGPRs(MaxWaves);
 
----------------
It would be OK to leave these as helpers in the reg info that call AMDGPU::IsaInfo


https://reviews.llvm.org/D28760