[PATCH] D133902: X86: Stop assigning register costs for longer encodings

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 30 17:33:12 PDT 2022


MatzeB added a comment.

I'm not sure I have 100% fixed it, we'll see :)

Also thinking about things right now, I realized that my change probably lowers the amount of times ragreedy asks for eviction decisions. I hope that doesn't reduce the quality of ML regalloc, maybe you can keep on eye on that on your end.

(I'm happy to revert if it decreases quality, though we have to find a better way than somewhat "arbitrary" CostPerUse / high register numbers triggering things long-term...)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133902



More information about the llvm-commits mailing list