[PATCH] D95762: AMDGPU: Don't consider global pressure when bundling soft clauses
Stanislav Mekhanoshin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 1 10:46:23 PST 2021
rampitec added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/SIFormMemoryClauses.cpp:249
+ bool IsVMEMClause) {
+ GCNRegPressure OldPressure = RPT.currentPressure();
+ unsigned OldOccupancy = OldPressure.getOccupancy(*ST);
----------------
I do not like copying the array. Maybe just expose proxy helpers getOccupancy(), getGurVGPRNum(), and getCurSGPRNum() from GCNRPTracker?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D95762/new/
https://reviews.llvm.org/D95762
More information about the llvm-commits
mailing list