[PATCH] D100904: [AMDGPU] Lower regbanks reassign threshold to 15000

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added a comment.

In D100904#2707767 <https://reviews.llvm.org/D100904#2707767>, @madhur13490 wrote:

> I feel we should probably discuss more about the optimal solution than the current one. I have a gut feeling that the problem pass is trying to solve is a flavor of graph coloring problem where we have to color the interference graph with K number of colors where K is the number of register banks. We may need to redo the interference graph in this pass and then attempt the coloring.

It would be much better to implement it inside the RA itself. But only if it is a real thing. The current pass can eliminate most of the conflicts. I yet want to see a confirmation it improves performance just anywhere and quantify it.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D100904/new/

https://reviews.llvm.org/D100904