[PATCH] D92669: [PGO] Adjust -vp-counters-per-site.

[Hiroshi Yamauchi via Phabricator via llvm-commits]

yamauchi created this revision.
yamauchi added a reviewer: davidxl.
Herald added subscribers: wenlei, hiraditya.
yamauchi requested review of this revision.
Herald added a project: LLVM.

Addressing clang bootstrap under the dynamic linking mode running out of static
allocation of value profile nodes, reported in D81682 <https://reviews.llvm.org/D81682>. The vnodes section size
goes from 1501k to 2171k (+670k bytes).


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D92669

Files:
  llvm/lib/Transforms/Instrumentation/InstrProfiling.cpp


Index: llvm/lib/Transforms/Instrumentation/InstrProfiling.cpp
===================================================================
--- llvm/lib/Transforms/Instrumentation/InstrProfiling.cpp
+++ llvm/lib/Transforms/Instrumentation/InstrProfiling.cpp
@@ -105,7 +105,7 @@
     // a very small percentage of value sites have non-zero targets, e.g, 1/30.
     // For those sites with non-zero profile, the average number of targets
     // is usually smaller than 2.
-    cl::init(1.0));
+    cl::init(1.5));
 
 cl::opt<bool> AtomicCounterUpdateAll(
     "instrprof-atomic-counter-update-all", cl::ZeroOrMore,


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D92669.309567.patch
Type: text/x-patch
Size: 604 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201204/01da0c8f/attachment.bin>