[PATCH] D37411: AMDGPU: Fix not accounting for tail call resource usage

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/AMDGPUAsmPrinter.cpp:503
   // count easily.
-  if (!FrameInfo.hasCalls()) {
+  if (!FrameInfo.hasCalls() && !FrameInfo.hasTailCall()) {
     MCPhysReg HighestVGPRReg = AMDGPU::NoRegister;
----------------
rampitec wrote:
> If it does not have calls it may not have tail calls. Am I missing something?
A tail call is not considered a call of hasCalls's purposes. If there is only a tail call it reports false.


https://reviews.llvm.org/D37411