This is an archive of the discontinued LLVM Phabricator instance.

Differential D83903

[PGO] Enable the extended value profile buckets for mem op sizes.
ClosedPublic

Authored by hjyamauchi on Jul 15 2020, 1:26 PM.

Details

Reviewers
davidxl
Commits
rG3e89cbf38e76: [PGO] Enable the extended value profile buckets for mem op sizes.
rG9f5d8e8a729d: [PGO] Enable the extended value profile buckets for mem op sizes.
Summary

Following up D81682 and enable the new, extended value profile buckets for mem
op sizes.

Diff Detail

Revision Contents

PathSize
llvm/
lib/
Transforms/
Instrumentation/
2 lines

Diff 279405

llvm/lib/Transforms/Instrumentation/InstrProfiling.cpp

Show First 20 Lines • Show All 72 Lines • ▼ Show 20 Linescl::opt<unsigned> MemOPSizeLarge(
cl::desc("Set large value thresthold in memory intrinsic size profiling. " cl::desc("Set large value thresthold in memory intrinsic size profiling. "
"Value of 0 disables the large value profiling."), "Value of 0 disables the large value profiling."),
cl::init(8192)); cl::init(8192));
cl::opt<bool> UseOldMemOpValueProf( cl::opt<bool> UseOldMemOpValueProf(
"use-old-memop-value-prof", "use-old-memop-value-prof",
cl::desc("Use the old memop value profiling buckets. This is " cl::desc("Use the old memop value profiling buckets. This is "
"transitional and to be removed after switching. "), "transitional and to be removed after switching. "),
cl::init(true)); cl::init(false));
namespace { namespace {
cl::opt<bool> DoHashBasedCounterSplit( cl::opt<bool> DoHashBasedCounterSplit(
"hash-based-counter-split", "hash-based-counter-split",
cl::desc("Rename counter variable of a comdat function based on cfg hash"), cl::desc("Rename counter variable of a comdat function based on cfg hash"),
cl::init(true)); cl::init(true));
▲ Show 20 Lines • Show All 1,057 Lines • Show Last 20 Lines