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

Handle unsigned integer constants greater than u32::MAX in codegen #1081

Merged
merged 1 commit into from Oct 13, 2017

Conversation

Projects
None yet
4 participants
@fitzgen
Copy link
Member

fitzgen commented Oct 13, 2017

We were not checking signed-ness and emitting the appropriate types.

Fixes #1040

r? @pepyakin

Handle unsigned integer constants greater than u32::MAX in codegen
We were not checking signed-ness and emitting the appropriate types.

Fixes #1040
@pepyakin

This comment has been minimized.

Copy link
Contributor

pepyakin commented Oct 13, 2017

@bors-servo r+

Awesome!

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Oct 13, 2017

📌 Commit 0068c1a has been approved by pepyakin

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Oct 13, 2017

⌛️ Testing commit 0068c1a with merge 8582a90...

bors-servo added a commit that referenced this pull request Oct 13, 2017

Auto merge of #1081 - fitzgen:issue-1040, r=pepyakin
Handle unsigned integer constants greater than u32::MAX in codegen

We were not checking signed-ness and emitting the appropriate types.

Fixes #1040

r? @pepyakin
@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Oct 13, 2017

☀️ Test successful - status-travis
Approved by: pepyakin
Pushing 8582a90 to master...

@bors-servo bors-servo merged commit 0068c1a into rust-lang:master Oct 13, 2017

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.