[PATCH] D63581: (Reland with changes) Adding a function for setting coverage output file.

Max Moroz via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 20 16:21:27 PDT 2019


Dor1s accepted this revision.
Dor1s added a comment.
This revision is now accepted and ready to land.

LGTM, thanks @sajjadm !



================
Comment at: compiler-rt/lib/profile/InstrProfilingFile.c:301
 
-  fclose(OutputFile);
+  if (OutputFile == getProfileFile()) {
+    fflush(OutputFile);
----------------
Looks like only this part has changed compared to https://reviews.llvm.org/D62541 + the tests.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63581





More information about the llvm-commits mailing list