[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