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

Remove SemLiterals #7

Open
Tracked by #8
liquidev opened this issue Sep 1, 2021 · 0 comments
Open
Tracked by #8

Remove SemLiterals #7

liquidev opened this issue Sep 1, 2021 · 0 comments

Comments

@liquidev
Copy link
Member

liquidev commented Sep 1, 2021

The lowering done by SemLiterals is better done in SemTypes, which has all the context about integer types it needs. With the current implementation the following is invalid:

var x: Size = 1

because 1 is inferred to be of type Int32, which cannot be converted to an unsigned type like Size.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant