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

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 4 10:35:17 PDT 2017


rampitec 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;
----------------
If it does not have calls it may not have tail calls. Am I missing something?


https://reviews.llvm.org/D37411





More information about the llvm-commits mailing list