Add path remapping with -coverage-prefix-map to coverage data#32416
Merged
varungandhi-apple merged 1 commit intoswiftlang:release/5.3from Sep 11, 2020
Merged
Add path remapping with -coverage-prefix-map to coverage data#32416varungandhi-apple merged 1 commit intoswiftlang:release/5.3from
varungandhi-apple merged 1 commit intoswiftlang:release/5.3from
Conversation
Previously the path to covered files in the __LLVM_COV / __llvm_covmap section were absolute. This made remote builds with coverage information difficult because all machines would have to have the same build root. This change uses the values for `-coverage-prefix-map` to remap files in the coverage info to relative paths. These paths work correctly with llvm-cov when it is run from the same source directory as the compilation, or from a different directory using the `-path-equivalence` argument. This is analogous to this change in clang https://reviews.llvm.org/D81122
keith
force-pushed
the
ks/cp-coverage-relative
branch
from
94ec20f to
c7d7b1e
Compare
Contributor
|
Previous history for context:
|
Contributor
|
@swift-ci test |
varungandhi-apple
approved these changes
Sep 11, 2020
DougGregor
approved these changes
Sep 11, 2020
Member
Author
|
Thanks!! |
AnthonyLatsis
added
swift 5.3
🍒 release cherry pick
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Previously the path to covered files in the __LLVM_COV / __llvm_covmap
section were absolute. This made remote builds with coverage information
difficult because all machines would have to have the same build root.
This change uses the values for
-coverage-prefix-mapto remap files inthe coverage info to relative paths. These paths work correctly with
llvm-cov when it is run from the same source directory as the
compilation, or from a different directory using the
-path-equivalenceargument.
This is analogous to this change in clang https://reviews.llvm.org/D81122