[PATCH] D81429: [COFF] Port CallGraphSort to COFF from ELF

Zequan Wu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 9 09:52:25 PDT 2020


zequanwu added inline comments.


================
Comment at: lld/COFF/CallGraphSort.cpp:198
+    // Don't consider merging if the edge is unlikely.
+    if (c.bestPred.from == -1 || c.bestPred.weight * 10 <= c.initialWeight)
+      continue;
----------------
MaskRay wrote:
> The coefficient is pretty arbitrary. Best if you can test which constant works well.
What is the tool I can use for testing this?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81429





More information about the llvm-commits mailing list