[PATCH] D23106: [Profile] introduce interface __llvm_profile_dump

David Li via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 8 10:26:40 PDT 2016


davidxl marked an inline comment as done.

================
Comment at: lib/profile/InstrProfiling.c:81
@@ -70,2 +80,3 @@
   }
+  ProfileDumped = 0;
 }
----------------
vsk wrote:
> Should probably also be cleared when llvm_prof_set_filename() or llvm_profile_merge_from_buffer() are called.
It is a mistake to not call reset counter before dumping again (due to double counting) regardless whether merging is used or not


https://reviews.llvm.org/D23106





More information about the llvm-commits mailing list