[llvm] Greedy: Make eviction broken hint cost use CopyCost units (PR #160084)
Philip Reames via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 29 09:21:01 PDT 2025
================
@@ -300,12 +300,14 @@ bool DefaultEvictionAdvisor::canEvictInterferenceBasedOnCost(
return false;
// We permit breaking cascades for urgent evictions. It should be the
// last resort, though, so make it really expensive.
- Cost.BrokenHints += 10;
+ Cost.BrokenHints += 10 * MRI->getRegClass(Intf->reg())->getCopyCost();
----------------
preames wrote:
Or we should explicitly use saturating arithmetic here. Either way, probably worth a revert and reapply with a fix.
https://github.com/llvm/llvm-project/pull/160084
More information about the llvm-commits
mailing list