[PATCH] D9009: Value profiling compiler-rt changes

[Betul Buyukkurt via llvm-commits]

betulb updated this revision to Diff 39981.
betulb added a comment.

In this revision:

- Each value site can write up to 255 value data elements to output file.
- The data type of NumValueSites[] has been changed from uint32_t to uint16_t
- Max value for the  value kind enum is stored in the header of the raw profile data file.
- PerSiteValueCounts are embedded into the same array with the actual value data.
- The __llvm_profile_data struct elements are ordered to make the NumValueSites[] the last member of the struct.

- Per review comments, ValueCounters member of the  __llvm_profile_data struct is modified during profile file write out to allow for the reader to read profile data file content in a uniform manner. This change however, needs to be revisited as it'd potentially break the buffer API usage, when the value profiling support is added there as well.


http://reviews.llvm.org/D9009

Files:
  lib/profile/InstrProfiling.c
  lib/profile/InstrProfiling.h
  lib/profile/InstrProfilingBuffer.c
  lib/profile/InstrProfilingFile.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D9009.39981.patch
Type: text/x-patch
Size: 12570 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151111/29c950bf/attachment.bin>