[PATCH] D102429: [CSSPGO] Update pseudo probe distribution factor based on inline context.

Hongtao Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 13 11:07:17 PDT 2021


hoy created this revision.
Herald added subscribers: dexonsmith, wenlei, hiraditya.
hoy requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

With prelink inlining, pseudo probes with same ID can come from different inline contexts. Such probes should not share samples and their factors should be fixed up separately.

I'm seeing 0.3% speedup for SPEC2017 overall. Benchmark 631.deepsjeng_s benefits the most, about 4%.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D102429

Files:
  llvm/include/llvm/IR/PseudoProbe.h
  llvm/include/llvm/Transforms/IPO/SampleProfileProbe.h
  llvm/lib/IR/PseudoProbe.cpp
  llvm/lib/Transforms/IPO/SampleProfileProbe.cpp
  llvm/test/Transforms/SampleProfile/pseudo-probe-update-2.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D102429.345225.patch
Type: text/x-patch
Size: 8809 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210513/06c3c412/attachment.bin>


More information about the llvm-commits mailing list