[PATCH] D67673: [PGO] Change hardcoded thresholds for cold/inlinehint to use summary

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 17 14:03:15 PDT 2019


davidxl added inline comments.


================
Comment at: lib/Transforms/Instrumentation/PGOInstrumentation.cpp:1626
 
+  // Add the profile summary (read from the header of the indexed summary) here
+  // so that we can use it below when reading counters (which checks if the
----------------
Conceptually, this should be set even earlier before getAnalysis at line 1754 below.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D67673





More information about the llvm-commits mailing list