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

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 29 17:08:03 PDT 2019


davidxl added inline comments.


================
Comment at: compiler-rt/test/profile/instrprof-set-file-object.c:5
+// RUN: test -f %t.file.profraw
+// RUN: llvm-profdata show %t.file.profraw
+// RUN: rm %t.file.profraw
----------------
Check the result of the dump.


================
Comment at: compiler-rt/test/profile/instrprof-set-file-object.c:7
+// RUN: rm %t.file.profraw
+#include <stdio.h>
+
----------------
Also add a test for merging. Run the test program sequentially and there is no need for locking in the test, but merging results need to be checked to correct.


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