Skip to content

SILGen: Remove triplesAreValidForZippering() assert #76276

New issue

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

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

Already on GitHub? # to your account

Conversation

tshortli
Copy link
Contributor

@tshortli tshortli commented Sep 5, 2024

This assert was correctly catching the fact that -target-variant is not being normalized at the same time as -target when building arm64e modules from swiftinterface. That should be fixed, but at the moment it isn't causing any concrete harm and the assertion fails when building against the SDKs included with the latest Xcode 16 betas.

Resolves rdar://133020098.

This assert was correctly catching the fact that `-target-variant` is not being
normalized at the same time as `-target` when building arm64e modules from
swiftinterface. That should be fixed, but at the moment it isn't causing any
concrete harm and the assertion fails when building against the SDKs included
with the latest Xcode 16 betas.

Resolves rdar://133020098.
@tshortli
Copy link
Contributor Author

tshortli commented Sep 5, 2024

@swift-ci please smoke test

@tshortli tshortli enabled auto-merge September 5, 2024 05:47
@tshortli
Copy link
Contributor Author

tshortli commented Sep 5, 2024

swiftlang/swift-testing#663

@swift-ci please smoke test Windows

@tshortli tshortli merged commit c04736f into swiftlang:main Sep 5, 2024
3 checks passed
@tshortli tshortli deleted the remove-valid-triples-for-zippering-assertion branch September 5, 2024 19:47
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant