[PATCH] D59143: [RegisterCoalescer] Limit the number of joins for large live interval with many valnos.

[Quentin Colombet via Phabricator via llvm-commits]

qcolombet accepted this revision.
qcolombet added a comment.
This revision is now accepted and ready to land.

Looks reasonable to me.



================
Comment at: lib/CodeGen/RegisterCoalescer.cpp:216
+    /// live intervals many times, we regard such live interval as high
+    /// cost.
+    bool isHighCostLiveInterval(LiveInterval &LI);
----------------
Maybe add that the high cost means will significantly impact compile time.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59143