[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