[PATCH] D66513: [SampleFDO] Add ExtBinary format to support extension of binary profile

[David Li via Phabricator via llvm-commits]

davidxl added a comment.

LLC miss info should be very sparse, so mixing it with LBR may not bring a lot of benefit.  Having mixure of different types of profile data can lead to combinatorial growth of the number of readers. As the number of profile types increases, it can cause the reader to be unmaintaintable.  That is why I suggest making it clean -- one type per section.

Regarding compact format. Does it mean when LLC misses are included, there is no way to use the compact representation?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D66513