[llvm] [memprof] Remove a stale comment in a unit test (PR #119060)
    via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Sat Dec  7 00:24:41 PST 2024
    
    
  
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-pgo
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
We've removed IndexedAllocationInfo::CallStack, so we don't need to
mention it.
---
Full diff: https://github.com/llvm/llvm-project/pull/119060.diff
1 Files Affected:
- (modified) llvm/unittests/ProfileData/InstrProfTest.cpp (-2) 
``````````diff
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);
``````````
</details>
https://github.com/llvm/llvm-project/pull/119060
    
    
More information about the llvm-commits
mailing list