Skip to content

[6.2][Concurrency] Hashable funcs should be inlinable for AsyncStream #81127

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

Conversation

ktoso
Copy link
Contributor

@ktoso ktoso commented Apr 28, 2025

Small review followup for #81064

The impls should be @inlinable.

Original PR: #81126
Radar: rdar://149914179

@ktoso ktoso requested a review from a team as a code owner April 28, 2025 02:04
@ktoso ktoso requested a review from DougGregor April 28, 2025 02:04
@ktoso ktoso changed the title [Concurrency] Hashable funcs should be inlinable for AsyncStream [6.2][Concurrency] Hashable funcs should be inlinable for AsyncStream Apr 28, 2025
@ktoso ktoso force-pushed the pick-wip-inlinable-for-hashable-asyncstream branch from 5442de0 to 5a1e66f Compare April 28, 2025 03:29
@ktoso
Copy link
Contributor Author

ktoso commented Apr 28, 2025

@swift-ci please test

@ktoso
Copy link
Contributor Author

ktoso commented Apr 28, 2025

@swift-ci please test

@ktoso ktoso force-pushed the pick-wip-inlinable-for-hashable-asyncstream branch from 5a1e66f to 1cfef25 Compare April 28, 2025 07:18
@ktoso
Copy link
Contributor Author

ktoso commented Apr 28, 2025

@swift-ci please test

@ktoso ktoso force-pushed the pick-wip-inlinable-for-hashable-asyncstream branch from 3387beb to ce2da1a Compare April 29, 2025 13:47
@ktoso
Copy link
Contributor Author

ktoso commented Apr 29, 2025

@swift-ci please test

@ktoso ktoso enabled auto-merge April 29, 2025 13:48
@ktoso
Copy link
Contributor Author

ktoso commented Apr 29, 2025

@swift-ci please test

@ktoso ktoso disabled auto-merge April 29, 2025 23:50
@ktoso ktoso enabled auto-merge (squash) April 29, 2025 23:50
@ktoso ktoso merged commit ec7ed9a into swiftlang:release/6.2 Apr 30, 2025
5 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.

3 participants