[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