[PATCH] D62541: Adding a function for setting coverage output file.

Sajjad Mirza via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 4 14:22:16 PDT 2019


sajjadm marked an inline comment as done.
sajjadm added inline comments.


================
Comment at: compiler-rt/lib/profile/InstrProfilingFile.c:306
 
-  fclose(OutputFile);
+#if !defined(_WIN32)
+  if (doMerging()) {
----------------
davidxl wrote:
> What is this for? Why locking the file again?
Good catch, that was supposed to be an unlock.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62541/new/

https://reviews.llvm.org/D62541





More information about the llvm-commits mailing list