[llvm] [RegisterCoalescer]: Try inflated RC for coalescing (PR #130870)
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 11 18:48:43 PDT 2025
https://github.com/arsenm commented:
I was working on the same thing a few weeks ago, and I think we need more targeted handling. This is too aggressively trying to recompute the register classes. In terms of patch splitting, I also think we should take each of these cases one at a time. I think the subregister insert case is the most likely to be profitable piece, so I think we should start there.
However, in terms of the AGPR-or-VGPR to AV_* class case, it will soon only be relevant for gfx908. I'm actively working on avoiding using AGPRs until the actual allocation forces it. I was also wondering if instead we should just be using the AV_* classes for all values in the first place.
I'll try to polish my patch into something postable.
https://github.com/llvm/llvm-project/pull/130870
More information about the llvm-commits
mailing list