[all-commits] [llvm/llvm-project] fa14fd: [CSSPGO][llvm-profgen] Change default cold thresho...

WenleiHe via All-commits all-commits at lists.llvm.org
Tue May 25 10:41:34 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: fa14fd30ce6775807463c71cbbaf58d4b30c5e9a
      https://github.com/llvm/llvm-project/commit/fa14fd30ce6775807463c71cbbaf58d4b30c5e9a
  Author: Wenlei He <aktoon at gmail.com>
  Date:   2021-05-25 (Tue, 25 May 2021)

  Changed paths:
    M llvm/test/tools/llvm-profgen/cs-extbinary.test
    M llvm/test/tools/llvm-profgen/fname-canonicalization.test
    M llvm/test/tools/llvm-profgen/inline-cs-dangling-pseudoprobe.test
    M llvm/test/tools/llvm-profgen/inline-cs-noprobe.test
    M llvm/test/tools/llvm-profgen/inline-cs-pseudoprobe.test
    M llvm/test/tools/llvm-profgen/merge-cold-profile.test
    M llvm/test/tools/llvm-profgen/noinline-cs-noprobe.test
    M llvm/test/tools/llvm-profgen/noinline-cs-pseudoprobe.test
    M llvm/test/tools/llvm-profgen/recursion-compression-noprobe.test
    M llvm/test/tools/llvm-profgen/recursion-compression-pseudoprobe.test
    M llvm/tools/llvm-profgen/ProfileGenerator.cpp

  Log Message:
  -----------
  [CSSPGO][llvm-profgen] Change default cold threshold for context merging

llvm-profgen uses profile summary based cold threshold to merge and trim cold context profile. This is to strike a good balance between profile size and performance.

We've been using 99.9% as the cutoff to save profile size without affecting performance. This change switch to use 99.9% instead of 99.9999% as default cold threshold cutoff for llvm-profgen.

Redundant switch csprof-cold-thres is also removed and tests cleaned up.

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




More information about the All-commits mailing list