Skip to content

[Driver] Force the test one-way-merge-module-fine to use emit-module #38966

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 2 commits into from
Sep 9, 2021

Conversation

xymus
Copy link
Contributor

@xymus xymus commented Aug 19, 2021

Update the test one-way-merge-module-fine.swift to force using merge-module with the new driver.

@xymus xymus requested a review from CodaFi August 19, 2021 22:34
@xymus
Copy link
Contributor Author

xymus commented Aug 19, 2021

@swift-ci Please smoke test

@xymus xymus force-pushed the one-way-emit-module branch from 3288f73 to 952a3a7 Compare August 20, 2021 00:15
@xymus
Copy link
Contributor Author

xymus commented Aug 20, 2021

@swift-ci Please smoke test

@xymus
Copy link
Contributor Author

xymus commented Aug 20, 2021

Tests are green with merge-module, let`s try with emit-module.

swiftlang/swift-driver#730
@swift-ci please smoke test

@xymus xymus force-pushed the one-way-emit-module branch from 952a3a7 to ee164a5 Compare September 3, 2021 22:25
@xymus
Copy link
Contributor Author

xymus commented Sep 3, 2021

swiftlang/swift-driver#730
@swift-ci please smoke test

@xymus xymus force-pushed the one-way-emit-module branch from ee164a5 to e5c8aa1 Compare September 7, 2021 23:45
As support, accept but ignore the -no-emit-module-separately flag in the
C++ driver.
@xymus xymus force-pushed the one-way-emit-module branch from e5c8aa1 to 3ddbb88 Compare September 8, 2021 17:20
@xymus
Copy link
Contributor Author

xymus commented Sep 8, 2021

Let's try something different for now.

@swift-ci Please smoke test

@xymus xymus changed the title [Driver] Update the test one-way-merge-module-fine to support emit-module-separately [Driver] Force the test one-way-merge-module-fine to use emit-module Sep 8, 2021
@xymus
Copy link
Contributor Author

xymus commented Sep 8, 2021

That looked good, let's try emit-module-separately now.

swiftlang/swift-driver#730
@swift-ci please smoke test

@xymus
Copy link
Contributor Author

xymus commented Sep 9, 2021

@swift-ci please smoke test macOS

@xymus xymus merged commit a144aa7 into swiftlang:main Sep 9, 2021
@xymus xymus deleted the one-way-emit-module branch September 9, 2021 15:11
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.

1 participant