[llvm] [nfc][ctx_prof] Rename `PGOContextualProfile` to `PGOCtxProfContext` (PR #102209)

Aiden Grossman via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 6 13:02:38 PDT 2024


================
@@ -24,16 +24,16 @@
 #include <vector>
 
 namespace llvm {
-/// The loaded contextual profile, suitable for mutation during IPO passes. We
-/// generally expect a fraction of counters and of callsites to be populated.
-/// We continue to model counters as vectors, but callsites are modeled as a map
-/// of a map. The expectation is that, typically, there is a small number of
-/// indirect targets (usually, 1 for direct calls); but potentially a large
-/// number of callsites, and, as inlining progresses, the callsite count of a
-/// caller will grow.
-class PGOContextualProfile final {
+/// A node (context) in the loaded contextual profile, suitable for mutation
+/// during IPO passes. We generally expect a fraction of counters and of
----------------
boomanaiden154 wrote:

Second of (at the end of the line) seems out of place?

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


More information about the llvm-commits mailing list