This is an archive of the discontinued LLVM Phabricator instance.

[llvm-profgen] Turn on preinliner by default
ClosedPublic

Authored by wenlei on Dec 14 2021, 5:12 PM.

Details

Summary

preinliner has been tuned on large server workloads and it's not ready to be turned on by default. this change also updates the thresholds based on tuning.

Diff Detail

Event Timeline

wenlei created this revision.Dec 14 2021, 5:12 PM
wenlei requested review of this revision.Dec 14 2021, 5:12 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 14 2021, 5:12 PM
hoy added inline comments.Dec 14 2021, 5:15 PM
llvm/tools/llvm-profgen/CSPreInliner.cpp
44

Should the default value be set to true?

wenlei updated this revision to Diff 394424.Dec 14 2021, 5:36 PM

Actually turn on preinliner by default..

wenlei added inline comments.Dec 14 2021, 5:37 PM
llvm/tools/llvm-profgen/CSPreInliner.cpp
44

Ops.. ofc you're right.. :)

wlei accepted this revision.Dec 14 2021, 5:37 PM

LGTM, thanks!

This revision is now accepted and ready to land.Dec 14 2021, 5:37 PM
hoy accepted this revision.Dec 14 2021, 5:39 PM

lgtm, thanks.

This revision was landed with ongoing or failed builds.Dec 14 2021, 5:47 PM
This revision was automatically updated to reflect the committed changes.