Skip to content

[5.7] Support hermetic indexing information #58946

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 19, 2022

Conversation

DavidGoldman
Copy link
Contributor

Cherry pick #42394 to release/5.7.

Swiftc port of swiftlang/llvm-project#4207.

This introduces a new flag, -file-prefix-map which can be used
instead of the existing -debug-prefix-map and -coverage-prefix-map
flags, and also remaps paths in index information currently.

This requires swiftlang/llvm-project#4688 to be merged at the same time (since they depend upon one another due to API changes).

Swiftc port of swiftlang/llvm-project#4207.

This introduces a new flag, `-file-prefix-map` which can be used
instead of the existing `-debug-prefix-map` and `-coverage-prefix-map`
flags, and also remaps paths in index information currently.
@DavidGoldman DavidGoldman requested a review from a team as a code owner May 17, 2022 19:13
@bnbarham bnbarham changed the title Cherry pick: Support hermetic indexing information [5.7] Support hermetic indexing information May 17, 2022
@bnbarham bnbarham requested a review from airspeedswift May 17, 2022 21:19
@bnbarham bnbarham added the r5.7 label May 17, 2022
@bnbarham
Copy link
Contributor

swiftlang/llvm-project#4688

@swift-ci please test

@bnbarham bnbarham merged commit 1b4cd82 into swiftlang:release/5.7 May 19, 2022
keith added a commit to bazelbuild/rules_swift that referenced this pull request Sep 14, 2022
This is a new flag in Swift 5.7, that mirrors the flag with the same
name in clang, meaning that it implies both -debug-prefix-map, and
-coverage-prefix map. Currently it is also the only flag that applies to
indexing info if you want to produce hermetic indexes. Ideally going
forward users should prefer this flag if they want their builds to be
hermetic instead of picking the other flags as needed, since this will
potentially also apply to new absolute paths in the future.

swiftlang/swift#58946
keith added a commit to bazelbuild/rules_swift that referenced this pull request Sep 14, 2022
This is a new flag in Swift 5.7, that mirrors the flag with the same
name in clang, meaning that it implies both -debug-prefix-map, and
-coverage-prefix map. Currently it is also the only flag that applies to
indexing info if you want to produce hermetic indexes. Ideally going
forward users should prefer this flag if they want their builds to be
hermetic instead of picking the other flags as needed, since this will
potentially also apply to new absolute paths in the future.

swiftlang/swift#58946
keith added a commit to bazelbuild/rules_swift that referenced this pull request Sep 15, 2022
This is a new flag in Swift 5.7, that mirrors the flag with the same
name in clang, meaning that it implies both -debug-prefix-map, and
-coverage-prefix map. Currently it is also the only flag that applies to
indexing info if you want to produce hermetic indexes. Ideally going
forward users should prefer this flag if they want their builds to be
hermetic instead of picking the other flags as needed, since this will
potentially also apply to new absolute paths in the future.

swiftlang/swift#58946
@AnthonyLatsis AnthonyLatsis added 🍒 release cherry pick Flag: Release branch cherry picks swift 5.7 labels Jan 9, 2023
tymurmustafaiev pushed a commit to tymurmustafaiev/rules_swift that referenced this pull request Jul 19, 2023
This is a new flag in Swift 5.7, that mirrors the flag with the same
name in clang, meaning that it implies both -debug-prefix-map, and
-coverage-prefix map. Currently it is also the only flag that applies to
indexing info if you want to produce hermetic indexes. Ideally going
forward users should prefer this flag if they want their builds to be
hermetic instead of picking the other flags as needed, since this will
potentially also apply to new absolute paths in the future.

swiftlang/swift#58946
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒 release cherry pick Flag: Release branch cherry picks swift 5.7
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants