[llvm] [AMDGPU] Constrain AV->VReg if we do not exceed RP thresholds (PR #150086)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 24 07:54:43 PDT 2025


arsenm wrote:

> The issue is that the register class priority is dependent upon the register pressure situation. If we need to use AGPRs, then it is best to assign AV last, otherwise, it is best to assign based on the bitwidth of the register. To address minor increase in register usage caused by #146606

Can you just drop the AV priorities to the bottom regardless of bitwidth?

> 
> A more direct solution is to provide a hook at some level in DefaultPriorityAdvisor::getPriority to allow dynamic RC priorites based on RP situation.

I don't think the scheduler should be touching register classes at all. It has impacts outside of the current scheduling region, and the scheduler should not be increasing constraints 

https://github.com/llvm/llvm-project/pull/150086


More information about the llvm-commits mailing list