[llvm] [CodeLayout] CDSortImpl: remove linear-time erase_value from mergeChains (PR #69276)

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 17 15:26:16 PDT 2023


================
@@ -1152,6 +1153,7 @@ class CDSortImpl {
       MergeGainT BestGain = BestEdge->getMergeGain();
       mergeChains(BestSrcChain, BestDstChain, BestGain.mergeOffset(),
                   BestGain.mergeType());
+      --NumHotChains;
----------------
MaskRay wrote:

Thanks for the suggestion. I'll optimize out `HotChains` and rename `NumHotChains` to `NumActiveChains`.

https://github.com/llvm/llvm-project/pull/69276


More information about the llvm-commits mailing list