[PATCH] D93254: [NFC][SampleFDO] Preparation to support multiple sections with the same type in ExtBinary format.

Mon Dec 14 21:55:24 PST 2020

wmi added a comment.

In D93254#2453854 <https://reviews.llvm.org/D93254#2453854>, @wenlei wrote:

>> To reduce the profile loading time during postlink phase when ThinLTO is enabled, we are going to split profile into two parts. One part contains profiles with inline instance and another part contains flattened profiles without inline instance in them.
>
> For the flattened profile, do you explicitly flatten part of raw profile, or it's just the portion of function profile that does not have any inlinee profile?

We explicitly flatten part of raw profile. We only keep inlinee profile for hot part. Flattening is necessary for us to reduce the size of the merged profile.

>> that could reduce the profile loading time for the large synthetic profile we use.
>
> Is that the partial profile that is a merged profile from a few workloads?

Yes, it is.

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93254/new/

https://reviews.llvm.org/D93254