[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