Skip to content

Conversation

@hamishknight
Copy link
Contributor

This wasn't a strict weak ordering, and it turns out violating that is undefined behavior which can result in buffer overruns.

This wasn't a strict weak ordering, and it turns out violating that
is undefined behavior which can result in buffer overruns.
@hamishknight
Copy link
Contributor Author

@swift-ci please smoke test

Copy link
Member

@rintaro rintaro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice find!

@hamishknight hamishknight enabled auto-merge July 9, 2025 20:24
@hamishknight hamishknight merged commit fd7dd78 into swiftlang:main Jul 10, 2025
3 checks passed
@hamishknight hamishknight deleted the sorted branch July 10, 2025 07:43
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.

2 participants