[PATCH] D100232: [Coverage] Support overriding compilation directory

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 9 14:32:37 PDT 2021


phosek created this revision.
phosek added reviewers: vsk, zequanwu.
Herald added a subscriber: hiraditya.
phosek requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

When making compilation relocatable, for example in distributed
compilation scenarios, we want to set compilation dir to a relative
value like `.` but this presents a problem when generating reports
because if the file path is relative as well, for example `..`, you
may end up writing files outside of the output directory.

This change introduces a flag that allows overriding the compilation
directory that's stored inside the profile with a different value that
is absolute.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D100232

Files:
  llvm/include/llvm/ProfileData/Coverage/CoverageMapping.h
  llvm/include/llvm/ProfileData/Coverage/CoverageMappingReader.h
  llvm/lib/ProfileData/Coverage/CoverageMapping.cpp
  llvm/lib/ProfileData/Coverage/CoverageMappingReader.cpp
  llvm/tools/llvm-cov/CodeCoverage.cpp
  llvm/tools/llvm-cov/CoverageViewOptions.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D100232.336565.patch
Type: text/x-patch
Size: 15878 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210409/d7581fe5/attachment.bin>


More information about the llvm-commits mailing list