[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