[PATCH] D16374: [sancov] generating html report on coverage dump

[Alexey Samsonov via llvm-commits]

samsonov added a comment.

Looks reasonsable to me. I'll let Kostya sign this off.


================
Comment at: test/asan/TestCases/Posix/coverage_html_report.cc:6
@@ +5,3 @@
+// RUN: cd %T/coverage_html_report
+// RUN: %env_asan_opts=coverage=1:verbosity=1:html_cov_report=1:sancov=%sancovcc %run %t 2>&1 | FileCheck %s --check-prefix=CHECK-main
+// RUN: ls *.html | FileCheck %s --check-prefix=CHECK-ls
----------------
Will this test work even if you don't provide sancov=%sancovcc explicitly?

================
Comment at: test/asan/TestCases/Posix/coverage_html_report.cc:24
@@ +23,2 @@
+// CHECK-main: html report generated to ./coverage_html_report.cc.tmp.[[PID]].html
+// CHECK-ls: coverage_html_report.cc.tmp.{{[0-9]+}}.html
----------------
Should this also be [[PID]].html ?


http://reviews.llvm.org/D16374