Skip to content

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants