[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