[llvm] [memprof] Add Version2 of the indexed MemProf format (PR #89100)
    Kazu Hirata via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Apr 18 12:39:20 PDT 2024
    
    
  
kazutakahirata wrote:
> Do you think we should add the v2 reader to the same patch so it can be tested as a round trip? Right now this patch doesn't have any tests.
The very first iteration has a smoke test `llvm/test/tools/llvm-profdata/memprof-merge-v0.test`, which is very weak.
The latest iteration adds a roundtrip unit test to `llvm/unittests/ProfileData/InstrProfTest.cpp`.  Please take a look.  Thanks!
https://github.com/llvm/llvm-project/pull/89100
    
    
More information about the llvm-commits
mailing list