Skip to content

System: remove unnecessary type casting#308

Merged
compnerd merged 1 commit into
apple:mainfrom
compnerd:typecast
Jun 11, 2026
Merged

System: remove unnecessary type casting#308
compnerd merged 1 commit into
apple:mainfrom
compnerd:typecast

Conversation

@compnerd

Copy link
Copy Markdown
Collaborator

The compiler now imports these as their canonical type, allowing us to directly use without the explicit type cast.

@milseman milseman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@milseman

milseman commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

@swift-ci please test

@compnerd

Copy link
Copy Markdown
Collaborator Author

@milseman is 6.3 the floor? if so, we will need to limit this until the floor is lifted, and only make the incorrect truncations explicit.

@compnerd

Copy link
Copy Markdown
Collaborator Author

@swift-ci please test

@compnerd compnerd force-pushed the typecast branch 3 times, most recently from 0f05243 to 7d24d78 Compare June 11, 2026 16:14
The compiler now imports these as their canonical type, allowing us to
directly use without the explicit type cast. Due to the floor of Swift
6.3, we must preserve the type cast values. Introduce an overlay to
explicitly cast the values in the older compilers.
@compnerd

Copy link
Copy Markdown
Collaborator Author

@swift-ci please test

@compnerd compnerd merged commit 8af20f0 into apple:main Jun 11, 2026
60 checks passed
@compnerd compnerd deleted the typecast branch June 11, 2026 20:16
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