[all-commits] [llvm/llvm-project] 6a74ad: [sancov] Accommodate sancov and coverage report se...

Vitaly Buka via All-commits all-commits at lists.llvm.org
Tue May 26 14:37:06 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 6a74ad6baad45b8572d196f7f290593ed62075b5
      https://github.com/llvm/llvm-project/commit/6a74ad6baad45b8572d196f7f290593ed62075b5
  Author: Vitaly Buka <vitalybuka at google.com>
  Date:   2020-05-26 (Tue, 26 May 2020)

  Changed paths:
    M llvm/test/tools/sancov/blacklist.test
    M llvm/test/tools/sancov/covered_functions.test
    M llvm/test/tools/sancov/merge.test
    M llvm/test/tools/sancov/not_covered_functions.test
    M llvm/test/tools/sancov/print.test
    M llvm/test/tools/sancov/stats.test
    M llvm/test/tools/sancov/symbolize.test
    M llvm/test/tools/sancov/symbolize_noskip_dead_files.test
    M llvm/test/tools/sancov/validation.test
    M llvm/tools/sancov/coverage-report-server.py
    M llvm/tools/sancov/sancov.cpp

  Log Message:
  -----------
  [sancov] Accommodate sancov and coverage report server for use under Windows

Summary:
This patch makes the following changes to SanCov and its complementary Python script in order to resolve issues pertaining to non-UNIX file paths in JSON symbolization information:
* Convert all paths to use forward slash.
* Update `coverage-report-server.py` to correctly handle paths to sources which contain spaces.
* Remove Linux platform restriction for all SanCov unit tests. All SanCov tests passed when ran on my local Windows machine.

Patch by Douglas Gliner.

Reviewers: kcc, filcab, phosek, morehouse, vitalybuka, metzman

Reviewed By: vitalybuka

Subscribers: vsk, Dor1s, llvm-commits

Tags: #sanitizers, #llvm

Differential Revision: https://reviews.llvm.org/D51018




More information about the All-commits mailing list