[PATCH] D31013: CodeGen: BlockPlacement: Reduce TriangleChainCount to 2

Kyle Butt via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 15 16:23:30 PDT 2017


iteratee created this revision.

This produces a 1% speedup on an important internal Google benchmark
(protocol buffers), with no other regressions in google or in the llvm
test-suite. Only 5 targets in the entire llvm test-suite are affected,
and on those 5 targets the size increase is 0.027%


Repository:
  rL LLVM

https://reviews.llvm.org/D31013

Files:
  lib/CodeGen/MachineBlockPlacement.cpp


Index: lib/CodeGen/MachineBlockPlacement.cpp
===================================================================
--- lib/CodeGen/MachineBlockPlacement.cpp
+++ lib/CodeGen/MachineBlockPlacement.cpp
@@ -149,7 +149,7 @@
     "triangle-chain-count",
     cl::desc("Number of triangle-shaped-CFG's that need to be in a row for the "
              "triangle tail duplication heuristic to kick in. 0 to disable."),
-    cl::init(3),
+    cl::init(2),
     cl::Hidden);
 
 extern cl::opt<unsigned> StaticLikelyProb;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D31013.91957.patch
Type: text/x-patch
Size: 508 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170315/39a6c261/attachment.bin>


More information about the llvm-commits mailing list