[llvm] 4acba06 - [memprof] Use a new constructor of IndexedAllocationInfo (NFC) (#116920)

via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 20 08:36:44 PST 2024


Author: Kazu Hirata
Date: 2024-11-20T08:36:40-08:00
New Revision: 4acba0697e7d8068927753f3bdabad478df91dc4

URL: https://github.com/llvm/llvm-project/commit/4acba0697e7d8068927753f3bdabad478df91dc4
DIFF: https://github.com/llvm/llvm-project/commit/4acba0697e7d8068927753f3bdabad478df91dc4.diff

LOG: [memprof] Use a new constructor of IndexedAllocationInfo (NFC) (#116920)

IndexedAllocationInfo now has a new constructor that allows us to omit
the inline call stack, which is going away soon.  This patch migrates
away from the old constructor.

Added: 
    

Modified: 
    llvm/unittests/Transforms/Instrumentation/MemProfUseTest.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/unittests/Transforms/Instrumentation/MemProfUseTest.cpp b/llvm/unittests/Transforms/Instrumentation/MemProfUseTest.cpp
index bf466345583322..2fa8610fcad4cc 100644
--- a/llvm/unittests/Transforms/Instrumentation/MemProfUseTest.cpp
+++ b/llvm/unittests/Transforms/Instrumentation/MemProfUseTest.cpp
@@ -323,7 +323,7 @@ makeRecordV2(std::initializer_list<::llvm::memprof::CallStackId> AllocFrames,
   for (const auto &CSId : AllocFrames) {
     // We don't populate IndexedAllocationInfo::CallStack because we use it only
     // in Version1.
-    MR.AllocSites.push_back({{}, CSId, Block, Schema});
+    MR.AllocSites.emplace_back(CSId, Block, Schema);
   }
   for (const auto &CSId : CallSiteFrames)
     MR.CallSiteIds.push_back(CSId);


        


More information about the llvm-commits mailing list