[llvm] [ctx_prof] Extend `WorkloadImportsManager` to use the contextual profile (PR #98682)
Mircea Trofin via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 17 13:30:26 PDT 2024
================
@@ -174,6 +174,10 @@ static cl::opt<std::string> WorkloadDefinitions(
"}"),
cl::Hidden);
+static cl::opt<std::string>
+ ContextualProfile("thinlto-pgo-ctx-prof",
----------------
mtrofin wrote:
We don't currently have a textual representation of the contextual profile, and the import manager is internal to this file, meaning we can't write a unittest. I'd do either, though, if the new functionality was more involved, but it's really just taking data from the ctx profile reader - which is tested - and filling a set with it.
I refactored a bit more to leave the specific code to a minimum.
https://github.com/llvm/llvm-project/pull/98682
More information about the llvm-commits
mailing list