[PATCH] D65471: [AMDGPU] Reserve all AGPRs on targets which do not have them
Stanislav Mekhanoshin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 21 11:48:18 PDT 2020
rampitec added inline comments.
================
Comment at: llvm/trunk/lib/Target/AMDGPU/SIRegisterInfo.cpp:223
+ // Reserve all the rest AGPRs if there are no instructions to use it.
+ if (!ST.hasMAIInsts()) {
----------------
arsenm wrote:
> Do we really need to do this? If there are no AGPRs, no AGPRs virtual registers should be created for this to come up
It is just a safety net. Somebody can use an inline asm with agprs (and sometimes do).
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65471/new/
https://reviews.llvm.org/D65471
More information about the llvm-commits
mailing list