[llvm] [ctx_prof] CtxProfAnalysis: populate module data (PR #102930)

Mircea Trofin via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 12 15:20:56 PDT 2024


================
@@ -24,5 +24,26 @@ class PGOCtxProfLoweringPass : public PassInfoMixin<PGOCtxProfLoweringPass> {
 
   PreservedAnalyses run(Module &M, ModuleAnalysisManager &MAM);
 };
+
+/// Assign a GUID to functions as metadata. GUID calculation takes linkage into
+/// account, which may change especially through and after thinlto. By
+/// pre-computing and assigning as metadata, this mechanism is resilient to such
+/// changes (as well as name changes e.g. suffix ".llvm." additions). It's
+/// arguably a much simpler mechanism than PGO's current GV-based one, and can
+/// be made available more broadly.
----------------
mtrofin wrote:

It should also change at that point because this whole thing would move out :) But removed that bit.

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


More information about the llvm-commits mailing list