Skip to content

[Frontend][Index] Add a new -index-unit-ouput-path and filelist equivalent to the frontend #36195

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

Conversation

nathawes
Copy link
Contributor

These new options mirror -o and -output-filelist and are used instead of those options to supply the output file path(s) to record in the index store. This is intended to allow sharing index data across builds in separate directories (so different -o values) that are otherwise equivalent as far as the index data is concerned (e.g. an ASAN build and a non-ASAN build) by supplying the same -index-unit-output-path for both.

Resolves rdar://problem/74816412

…o the frontend

These new options mirror -o and -output-filelist and are used instead
of those options to supply the output file path(s) to record in the
index store. This is intended to allow sharing index data across
builds in separate directories that are otherwise equivalent as far
as the index data is concered (e.g. an ASAN build and a non-ASAN build)
by supplying the same -index-unit-output-path for both.

Resolves rdar://problem/74816412
@nathawes nathawes requested a review from akyrtzi February 27, 2021 03:08
@nathawes
Copy link
Contributor Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 821345c

@nathawes
Copy link
Contributor Author

@swift-ci please test macOS

@nathawes
Copy link
Contributor Author

@swift-ci please test Windows

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 821345c

@nathawes
Copy link
Contributor Author

nathawes commented Mar 2, 2021

@swift-ci please test macOS

@nathawes
Copy link
Contributor Author

nathawes commented Mar 2, 2021

@swift-ci please test Windows

1 similar comment
@nathawes
Copy link
Contributor Author

nathawes commented Mar 2, 2021

@swift-ci please test Windows

@swift-ci
Copy link
Contributor

swift-ci commented Mar 2, 2021

Build failed
Swift Test OS X Platform
Git Sha - 821345c

@nathawes
Copy link
Contributor Author

nathawes commented Mar 2, 2021

@swift-ci please smoke test macOS

@akyrtzi
Copy link
Contributor

akyrtzi commented Mar 2, 2021

@swift-ci Please test Windows platform

@akyrtzi
Copy link
Contributor

akyrtzi commented Mar 2, 2021

@swift-ci smoke test OS X platform

1 similar comment
@akyrtzi
Copy link
Contributor

akyrtzi commented Mar 2, 2021

@swift-ci smoke test OS X platform

@nathawes nathawes merged commit 81db4cf into swiftlang:main Mar 2, 2021
@nathawes nathawes deleted the add-index-unit-output-path-frontend branch March 2, 2021 22:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants