Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ir: Handle *const T at the codegen level. #1312

Merged
merged 4 commits into from
May 16, 2018

Conversation

emilio
Copy link
Contributor

@emilio emilio commented May 14, 2018

Followup to #1311.

@emilio
Copy link
Contributor Author

emilio commented May 14, 2018

r? @fitzgen

Copy link
Member

@fitzgen fitzgen left a comment

Choose a reason for hiding this comment

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

Awesome!

Is this purely a refactoring, or are there new cases where we can Do the Right Thing? If the latter, then we should add some more test cases 👍

@emilio
Copy link
Contributor Author

emilio commented May 15, 2018

I had a feeling this may fix some odd cases of mixing const and non-const pointer... Since you insisted it I found a test-case and I just added it :)

@bors-servo r=fitzgen

@bors-servo
Copy link

📌 Commit 716d53b has been approved by fitzgen

@bors-servo
Copy link

⌛ Testing commit 716d53b with merge d6d097c...

bors-servo pushed a commit that referenced this pull request May 15, 2018
ir: Handle *const T at the codegen level.

Followup to #1311.
@bors-servo
Copy link

☀️ Test successful - status-travis
Approved by: fitzgen
Pushing d6d097c to master...

@bors-servo bors-servo merged commit 716d53b into rust-lang:master May 16, 2018
@cynecx
Copy link
Contributor

cynecx commented May 17, 2018

Thanks for all the effort! However, I am wondering when we can expect a version bump with the recent changes. sodiumoxide/sodiumoxide#208 is actually waiting for an upstream release with these changes.

@emilio
Copy link
Contributor Author

emilio commented May 18, 2018 via email

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants