[PATCH] D44847: [profile] Fix value profile runtime merging issues

Rong Xu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 23 14:48:35 PDT 2018


xur created this revision.
xur added a reviewer: davidxl.

This patch fixed two issues:
(1) value profile merging did not worked as the strong definition of the merge hook function never worked. We were not doing the merging of value profile. This patch remove the weak attribute and assign the hook dynamically. 
(2) in case value profile is not merged, truncated the file so that we don't have garbage in the end of the profile.


https://reviews.llvm.org/D44847

Files:
  lib/profile/InstrProfilingFile.c
  lib/profile/InstrProfilingMerge.c
  lib/profile/InstrProfilingMergeFile.c
  test/profile/instrprof-value-merge.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D44847.139655.patch
Type: text/x-patch
Size: 4172 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180323/cda877d5/attachment.bin>


More information about the llvm-commits mailing list