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

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 23 14:56:14 PDT 2018


davidxl added inline comments.


================
Comment at: lib/profile/InstrProfilingFile.c:230
+  // (which can leave garbage at the end of profile file).
+  if (!VPMergeHook)
+    truncate(ProfileFileName, 0);
----------------
non value profiling data has fixed size, so why is this needed?


https://reviews.llvm.org/D44847





More information about the llvm-commits mailing list