[llvm] [AMDGPU] Optionally Use GCNRPTrackers during scheduling (PR #93090)
Jeffrey Byrnes via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 27 15:22:16 PDT 2024
jrbyrnes wrote:
Added several tests based on the more significant lit changes.
Since this is a change which impacts scheduling heuristics, the changes are a mix of regressions and improvements. I've analyzed and included all the significant regressions, and added comments / fixmes to expplain the situation. The causes are: 1. not tracking PhysRegs, 2. the scheduler impoving RP in initial stage and not triggering high-RP stage, and 3. the scheduler being overly sensitive to minor changes in RP, and not being willing to incur a small penalty to enable scheduling of RP reducing instructions.
These will be addresed in a follow-up PR which enables the GCN RP Tracker features by default.
I've also included several tests where things go as expected -- more accurate RP Tracking leads to reductions in RP and, ultimately, improving occupancy
https://github.com/llvm/llvm-project/pull/93090
More information about the llvm-commits
mailing list