Page MenuHomePhabricator

[utils] Test the code coverage regression-checking script
AbandonedPublic

Authored by vsk on Jul 22 2016, 11:30 AM.

Details

Summary

Make sure that it can parse llvm-cov report output, so that the script doesn't break when we update llvm-cov.

Diff Detail

Event Timeline

vsk updated this revision to Diff 65115.Jul 22 2016, 11:30 AM
vsk retitled this revision from to [utils] Test the code coverage regression-checking script.
vsk updated this object.
vsk added reviewers: MaggieYi, silvas, ikudrin.
vsk added a subscriber: llvm-commits.
MaggieYi edited edge metadata.Jul 24 2016, 1:49 AM

Thanks for adding the test.

I have built the test in Windows Visual Studio and it failed due to that the environment variable (LLVM_SRC_ROOT) isn’t set in Visual Studio.

vsk added a comment.Jul 25 2016, 12:59 PM

Thanks for adding the test.

I have built the test in Windows Visual Studio and it failed due to that the environment variable (LLVM_SRC_ROOT) isn’t set in Visual Studio.

Could you share more details about how you ran the test? I'm having a hard time understanding the failure because LLVM_SRC_ROOT is set in test/lit.cfg. That environment variable should be set on Windows builders if the test is run with llvm-lit test/tools/llvm-cov/report.cpp.

In D22692#495195, @vsk wrote:

Could you share more details about how you ran the test? I'm having a hard time understanding the failure because LLVM_SRC_ROOT is set in test/lit.cfg. That environment variable should be set on Windows builders if the test is run with llvm-lit test/tools/llvm-cov/report.cpp.

I built all of LLVM projects and ran the regression tests on Windows using Visual Studio, which are mentioned in http://clang.llvm.org/get_started.html.

  1. Check out LLVM/Clang;
  2. Run CMake to generate the Visual Studio solution and project files: % mkdir build % cd build % cmake -G "Visual Studio 12" ..\llvm An LLVM.sln file is created in the build directory.
  3. Open LLVM.sln in Visual Studio and build the "ALL_BUILD" project to build everything.
  4. Build “check-all” project to run the regression tests. The report.cpp test failed during this step.
vsk abandoned this revision.Sep 18 2016, 5:47 PM