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

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 19 15:51:25 PDT 2021


davidxl accepted this revision.
davidxl added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/lib/ProfileData/InstrProfReader.cpp:537
+      return make_error<InstrProfError>(instrprof_error::malformed);
+  }
+
----------------
When there is no binary ids to be print, perhaps output a line with 'None' instead of empty (which looks like the output is truncated.


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