Skip to content

Allow enabling embedded Swift without WMO when not generating SIL #70612

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
Jul 16, 2024

Conversation

owenv
Copy link
Contributor

@owenv owenv commented Dec 23, 2023

This allows modes like -index-file to work the same way they do when not using embedded Swift

@owenv
Copy link
Contributor Author

owenv commented Dec 23, 2023

swiftlang/swift-driver#1513
@swift-ci please build toolchain macOS

@rauhul
Copy link
Member

rauhul commented Jul 15, 2024

I'll defer to @kubamracek about correctness, but I'm very thankful to see improvements on this front :)

@kubamracek
Copy link
Contributor

kubamracek commented Jul 15, 2024

This is basically part of what I think we said with @bnbarham and @ahoppen we need to do. The other part is silencing PerfDiags errors when in non-WMO mode.

@bnbarham
Copy link
Contributor

Yeah, this is definitely one part, thanks @owenv 🙇! Mind cherry-picking it too?

@bnbarham
Copy link
Contributor

Dec 23, 2023

:O

@owenv owenv force-pushed the owenv/embedded-typecheck-without-wmo branch from aaa2085 to cd13467 Compare July 15, 2024 21:45
@owenv owenv requested review from artemcm and tshortli as code owners July 15, 2024 21:45
@owenv
Copy link
Contributor Author

owenv commented Jul 15, 2024

@bnbarham yeah, I wrote this over the holiday and then forgot about it for a bit. Let's make sure everything is still working after a rebase

@swift-ci please smoke test

@owenv
Copy link
Contributor Author

owenv commented Jul 15, 2024

@swift-ci please smoke test Linux

@owenv
Copy link
Contributor Author

owenv commented Jul 15, 2024

@swift-ci please smoke test macOS

@owenv owenv force-pushed the owenv/embedded-typecheck-without-wmo branch from cd13467 to 765deb1 Compare July 15, 2024 23:43
@owenv
Copy link
Contributor Author

owenv commented Jul 15, 2024

@swift-ci please smoke test

@owenv owenv force-pushed the owenv/embedded-typecheck-without-wmo branch from 765deb1 to f2f5384 Compare July 15, 2024 23:44
@owenv
Copy link
Contributor Author

owenv commented Jul 15, 2024

@swift-ci please smoke test

This allows modes like -index-file to work the same way they do when not using embedded Swift
@owenv owenv force-pushed the owenv/embedded-typecheck-without-wmo branch from f2f5384 to 5490b44 Compare July 16, 2024 00:49
@owenv
Copy link
Contributor Author

owenv commented Jul 16, 2024

@swift-ci please smoke test

@owenv owenv merged commit e4b56c7 into swiftlang:main Jul 16, 2024
3 checks passed
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.

4 participants