[llvm] [Profile] Add binary profile correlation to offload profile metadata at runtime. (PR #69493)

David Li via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 19 13:13:06 PDT 2023


================
@@ -98,13 +99,16 @@ extern cl::opt<bool> PrintPipelinePasses;
 static cl::opt<bool> ClSanitizeOnOptimizerEarlyEP(
     "sanitizer-early-opt-ep", cl::Optional,
     cl::desc("Insert sanitizers on OptimizerEarlyEP."), cl::init(false));
-}
+
+extern cl::opt<InstrProfCorrelator::ProfCorrelatorKind> ProfileCorrelate;
----------------
david-xl wrote:

No, what I meant is in the refactoring/preparation patch, change debug-info-correlate into profile-correlate=debug. The none kind is still kept. 

In your next patch to implement binary correlation, add the binary kind.

The goal for the preparation/refactoring patch is to minimize the diff in the second patch implementing the feature so that reviewer can have more focus on the functional diffs.

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


More information about the llvm-commits mailing list