Skip to content

stdlib: Fix more typed throws and non-copyable generics condfails #73306

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
Apr 27, 2024

Conversation

tshortli
Copy link
Contributor

There were two categories of problem for older compilers that consume the stdlib .swiftinterface:

  • The case .some(borrowing x) syntax isn't accepted; use _borrowing
  • Various functions that addopted ~Copyable generic constraints conflict with the @usableFromInline ABI stubs that were left in to preserve compatibility when the functions are printed with ~Copyable constraints stripped. The stubs need to have different names to get out of the way.

Fixes fallout from #73045.

Resolves rdar://127132742

There were two categories of problem for older compilers that consume the
stdlib `.swiftinterface`:

- The `case .some(borrowing x)` syntax isn't accepted; use `_borrowing`
- Various functions that addopted `~Copyable` generic constraints conflict with
  the `@usableFromInline` ABI stubs that were left in to preserve compatibility
  when the functions are printed with `~Copyable` constraints stripped. The stubs
  need to have different names to get out of the way.

Resolves rdar://127132742
@tshortli tshortli requested a review from a team as a code owner April 26, 2024 23:38
@tshortli
Copy link
Contributor Author

@swift-ci please test

@tshortli
Copy link
Contributor Author

@swift-ci please test Windows

@tshortli tshortli merged commit 2507583 into swiftlang:main Apr 27, 2024
@tshortli tshortli deleted the the-condfails-never-end branch April 27, 2024 23:07
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