[llvm] 427172a - [memprof] Remove a stale comment in a unit test (#119060)

via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 7 10:41:30 PST 2024


Author: Kazu Hirata
Date: 2024-12-07T10:41:26-08:00
New Revision: 427172a861f6a1ff1d00771b896d2b2a2ac34494

URL: https://github.com/llvm/llvm-project/commit/427172a861f6a1ff1d00771b896d2b2a2ac34494
DIFF: https://github.com/llvm/llvm-project/commit/427172a861f6a1ff1d00771b896d2b2a2ac34494.diff

LOG: [memprof] Remove a stale comment in a unit test (#119060)

We've removed IndexedAllocationInfo::CallStack, so we don't need to
mention it.

Added: 
    

Modified: 
    llvm/unittests/ProfileData/InstrProfTest.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/unittests/ProfileData/InstrProfTest.cpp b/llvm/unittests/ProfileData/InstrProfTest.cpp
index 273b89ca26aa98..49a186cc98df28 100644
--- a/llvm/unittests/ProfileData/InstrProfTest.cpp
+++ b/llvm/unittests/ProfileData/InstrProfTest.cpp
@@ -401,8 +401,6 @@ makeRecordV2(std::initializer_list<::llvm::memprof::CallStackId> AllocFrames,
              const MemInfoBlock &Block, const memprof::MemProfSchema &Schema) {
   llvm::memprof::IndexedMemProfRecord MR;
   for (const auto &CSId : AllocFrames)
-    // We don't populate IndexedAllocationInfo::CallStack because we use it only
-    // in Version1.
     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