[compiler-rt] [llvm] [ctxprof] Make ContextRoot an implementation detail (PR #131416)

[Snehasish Kumar via llvm-commits]

================
@@ -246,23 +234,23 @@ bool CtxInstrumentationLowerer::lowerFunction(Function &F) {
            ArrayType::get(Builder.getPtrTy(), NumCallsites)});
       // Figure out which way we obtain the context object for this function -
       // if it's an entrypoint, then we call StartCtx, otherwise GetCtx. In the
-      // former case, we also set TheRootContext since we need to release it
+      // former case, we also set TheRootFuctionData since we need to release it
       // at the end (plus it can be used to know if we have an entrypoint or a
       // regular function)
-      auto Iter = ContextRootMap.find(&F);
-      if (Iter != ContextRootMap.end()) {
-        TheRootContext = Iter->second;
+      // Make up a compact name, these names end up taking up a lot of space
----------------
snehasish wrote:

Is this comment out of date? Or are you referring to the fact that not specifying a name explicitly makes up a compact enough name?

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