[PATCH] D94435: [SampleFDO] Add the support to split the function profiles with context into separate sections.
Wenlei He via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 12 23:43:04 PST 2021
wenlei added a comment.
Thanks for the change.
> If the profile is large, it is better to split the profile into two parts, one with context and one without, so the profile reading in postlink phase only has to read the part with context.
I guess the speed up is more visible with merged partial profile with explicit profile flattening? or did you see it speed up FDO with regular full profile too? For a regular FDO profile, I'd expect most function profiles to have some context except for really small functions..
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D94435/new/
https://reviews.llvm.org/D94435
More information about the llvm-commits
mailing list