[llvm] [MemProf] Optionally save context size info on largest cold allocations (PR #142507)

[Snehasish Kumar via llvm-commits]

================
@@ -168,7 +187,8 @@ void CallStackTrie::addCallStack(MDNode *MIB) {
 static MDNode *createMIBNode(LLVMContext &Ctx, ArrayRef<uint64_t> MIBCallStack,
                              AllocationType AllocType,
                              ArrayRef<ContextTotalSize> ContextSizeInfo,
-                             uint64_t &TotalBytes, uint64_t &ColdBytes) {
+                             uint64_t &TotalBytes, uint64_t &ColdBytes,
----------------
snehasish wrote:

It's a bit strange to have output params (Total bytes and ColdBytes) appear before MaxColdSize which is an input. Maybe change the order to have the inputs first? Also const for MaxColdSize.

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