[PATCH] D102039: [profile] Add binary id into profiles

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 25 11:37:56 PDT 2021


davidxl added a comment.

I am less concerned about raw profile format change (the version still needs to be bumped), but for the indexed format, the version needs to be bumped and it needs to guarantee that format is backward compatible. The patch does not seem to handle it.

Is it necessary to have binary id support in the indexed format?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102039



More information about the llvm-commits mailing list