[PATCH] D105248: [CSSPGO] Do not import pseudo probe desc in thinLTO

Hongtao Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 1 13:10:40 PDT 2021


hoy added a comment.

Verified that for SPEC, the generated profiles w/ and w/o this changes are very close. The runtime performance of pass2 has no difference two. Regarding the pseudo probe sections, the .pseudo_probe section is exactly the same for all benchmarks. The .pseudo_probe_desc sections are mostly the same, except for a couple benchmarks, for which the probe decoding gives exact result. There might be a glitch in setting the comdat group name for duplicated functions, like the ones with `.llvm.` suffix, or an issue of our internal change thinlto dedup work.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105248



More information about the llvm-commits mailing list