This is an archive of the discontinued LLVM Phabricator instance.

Increase the ImportHotMultiplier to 10.0
ClosedPublic

Authored by danielcdh on Jul 27 2017, 5:18 PM.

Reviewers

davidxl
tejohnson

Commits

Summary

The original 3.0 hot mupltiplier is too small, and would prevent hot callsites from being inline. This patch increases the hot multilier to 10.0

Build Status

Buildable 8671
Build 8671: arc lint + arc unit

LGTM

This revision is now accepted and ready to land.Jul 27 2017, 5:54 PM

Path

Size

lib/

Transforms/

IPO/

2 lines

	Show First 20 Lines • Show All 55 Lines • ▼ Show 20 Lines
	static cl::opt<float> ImportHotInstrFactor(			static cl::opt<float> ImportHotInstrFactor(
	"import-hot-evolution-factor", cl::init(1.0), cl::Hidden,			"import-hot-evolution-factor", cl::init(1.0), cl::Hidden,
	cl::value_desc("x"),			cl::value_desc("x"),
	cl::desc("As we import functions called from hot callsite, multiply the "			cl::desc("As we import functions called from hot callsite, multiply the "
	"`import-instr-limit` threshold by this factor "			"`import-instr-limit` threshold by this factor "
	"before processing newly imported functions"));			"before processing newly imported functions"));

	static cl::opt<float> ImportHotMultiplier(			static cl::opt<float> ImportHotMultiplier(
	"import-hot-multiplier", cl::init(3.0), cl::Hidden, cl::value_desc("x"),			"import-hot-multiplier", cl::init(10.0), cl::Hidden, cl::value_desc("x"),
	cl::desc("Multiply the `import-instr-limit` threshold for hot callsites"));			cl::desc("Multiply the `import-instr-limit` threshold for hot callsites"));

	static cl::opt<float> ImportCriticalMultiplier(			static cl::opt<float> ImportCriticalMultiplier(
	"import-critical-multiplier", cl::init(100.0), cl::Hidden,			"import-critical-multiplier", cl::init(100.0), cl::Hidden,
	cl::value_desc("x"),			cl::value_desc("x"),
	cl::desc(			cl::desc(
	"Multiply the `import-instr-limit` threshold for critical callsites"));			"Multiply the `import-instr-limit` threshold for critical callsites"));

	▲ Show 20 Lines • Show All 805 Lines • Show Last 20 Lines