[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