[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