[PATCH] D107143: [profile] Fix merging with binary IDs

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 30 02:43:02 PDT 2021


phosek created this revision.
phosek added reviewers: gulfem, hans, davidxl.
Herald added a subscriber: hiraditya.
phosek requested review of this revision.
Herald added projects: Sanitizers, LLVM.
Herald added subscribers: llvm-commits, Sanitizers.

This fixes support for merging profiles which broke as a consequence
of e50a38840dc3db5813f74b1cd2e10e6d984d0e67 <https://reviews.llvm.org/rGe50a38840dc3db5813f74b1cd2e10e6d984d0e67>. Additionally, it also
moves the binary IDs size field after version field in the header
as was suggested in the review.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D107143

Files:
  compiler-rt/include/profile/InstrProfData.inc
  compiler-rt/lib/profile/InstrProfilingBuffer.c
  compiler-rt/lib/profile/InstrProfilingMerge.c
  compiler-rt/test/profile/Linux/binary-id.c
  llvm/include/llvm/ProfileData/InstrProfData.inc
  llvm/lib/ProfileData/InstrProfReader.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D107143.363008.patch
Type: text/x-patch
Size: 6485 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210730/563fd42d/attachment-0001.bin>


More information about the llvm-commits mailing list