[PATCH] D113785: [llvm-profgen] Trim cold function profiles for non-CS AutoFDO

Lei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 30 12:14:05 PST 2021


wlei added inline comments.


================
Comment at: llvm/test/tools/llvm-profgen/inline-noprobe2.test:111
+;CHECK-TRIM-COLD-NOT: quick_sort:83:25
+;CHECK-TRIM-COLD-NOT: main:52:0
+
----------------
> None of the tests cover trim vs no-trim difference, it'd be good to have one, so we check trimming is actually making a difference.

@wenlei  We do have a test case for the trimming here which checked the function won't appear if its total sample is smaller than `--profile-summary-cold-count=100`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113785



More information about the llvm-commits mailing list