[all-commits] [llvm/llvm-project] 97e2ae: [AutoFDO] Use getHeadSamplesEstimate instead of ge...

ictwanglei via All-commits all-commits at lists.llvm.org
Thu Dec 15 11:22:04 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 97e2aeab71c3acfad9dc9df000c88c78686d8092
      https://github.com/llvm/llvm-project/commit/97e2aeab71c3acfad9dc9df000c88c78686d8092
  Author: wlei <wlei at fb.com>
  Date:   2022-12-15 (Thu, 15 Dec 2022)

  Changed paths:
    M llvm/lib/Transforms/IPO/SampleProfile.cpp
    M llvm/test/Transforms/SampleProfile/profile-mismatch.ll
    M llvm/test/Transforms/SampleProfile/pseudo-probe-profile-mismatch.ll

  Log Message:
  -----------
  [AutoFDO] Use getHeadSamplesEstimate instead of getTotalSamples to compute profile callsite staleness

Fix two issues for profile staleness report.

1) It should be more accurate to use the sum of all entry count(`getHeadSamplesEstimate`) for the callsite samples than the total samples, since even the top-level callsite is mismatched, it does affect the inlining but it can still be merged into base profile and used later.

2) I accidentally missed to persist the num of mismatched callsite into binary.

Also added the asm testing to test the decoding of the section.

Reviewed By: hoy, wenlei

Differential Revision: https://reviews.llvm.org/D140063




More information about the All-commits mailing list