[PATCH] D104477: [CSSPGO] Undoing the concept of dangling pseudo probe

Fri Jun 18 13:45:17 PDT 2021

hoy added inline comments.

================
Comment at: llvm/test/tools/llvm-profgen/merge-cold-profile.test:25
 ; CHECK-NEXT: 8: 1 fa:1
-; CHECK-NEXT: !CFGChecksum: 120515930909
 ; CHECK-NEXT: !Attributes: 0
----------------
wenlei wrote:
> I thought the changes for removing dangling probes are later, and checksum is computed earlier? What triggered cfg checksum changes? 
It' probably due to different version of compilers used to build the test binary. Previous compiler didn't include number of direct calls in CFGChecksum.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D104477/new/

https://reviews.llvm.org/D104477