Skip to content
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

Add prelude support for implicit conversion between integer literals and sized integer types. #4525

Merged
merged 3 commits into from
Nov 15, 2024

Conversation

zygoloid
Copy link
Contributor

In preparation for changing integer literals to be of IntLiteral type.
Conversions from the integer literal type are only permitted when the
value fits within the destination type.

For now, if conversion cannot be checked because the source value is a
symbolic constant, produce a symbolic constant representing the
conversion rather than rejecting it.

@zygoloid

This comment was marked as resolved.

sized integer types.

In preparation for changing integer literals to be of `IntLiteral` type.
Conversions from the integer literal type are only permitted when the
value fits within the destination type.

For now, if conversion cannot be checked because the source value is a
symbolic constant, produce a symbolic constant representing the
conversion rather than rejecting it.
@zygoloid zygoloid force-pushed the toolchain-symbolic-convert branch from ada6eb9 to 4967122 Compare November 14, 2024 00:46
Don't hold a Function& after it may have been invalidated by importing.
@zygoloid zygoloid requested a review from josh11b November 14, 2024 22:03
@josh11b josh11b added this pull request to the merge queue Nov 14, 2024
@jonmeow jonmeow removed this pull request from the merge queue due to a manual request Nov 14, 2024
@jonmeow
Copy link
Contributor

jonmeow commented Nov 14, 2024

Interrupting the merge on this, @zygoloid can you just check due to the delta size, similar to other PRs?

@zygoloid
Copy link
Contributor Author

Yeah, let's decide on #4534 first, that should reduce the delta a lot here if we go with it.

@zygoloid zygoloid enabled auto-merge November 15, 2024 21:41
@zygoloid zygoloid added this pull request to the merge queue Nov 15, 2024
Merged via the queue into carbon-language:trunk with commit ff53030 Nov 15, 2024
8 checks passed
@zygoloid zygoloid deleted the toolchain-symbolic-convert branch November 15, 2024 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants