[llvm] [MemProf] Fix reporting with -memprof-matching-cold-threshold (PR #173327)
Teresa Johnson via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 23 07:32:57 PST 2025
teresajohnson wrote:
> Is it possible to add the allocation type to contexttotalsize struct instead of creating a new pair?
I didn't for a couple reasons. For one, the ContextTotalSize data structure is used other places, such as during the LTO step. And secondly, due to context trimming a summarized context with its allocation type can correspond to multiple context/size pairs. For the Trie during matching, the redundancy (in the new vector of pairs) is ok from a convenience standpoint, but during LTO I don't want to carry around extra memory.
https://github.com/llvm/llvm-project/pull/173327
More information about the llvm-commits
mailing list