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

codegen: Explicitly align unions if needed. #1394

Merged
merged 1 commit into from
Sep 21, 2018
Merged

Conversation

emilio
Copy link
Contributor

@emilio emilio commented Sep 20, 2018

Also adds a test header I missed from the previous PR.

Fixes #1393

Also adds a test header I missed from the previous PR.

Fixes rust-lang#1393
@highfive
Copy link

warning Warning warning

  • These commits modify unsafe code. Please review it carefully!

@emilio
Copy link
Contributor Author

emilio commented Sep 20, 2018

r? @fitzgen

@fitzgen
Copy link
Member

fitzgen commented Sep 21, 2018

@bors-servo r+

@bors-servo
Copy link

📌 Commit cb12a31 has been approved by fitzgen

@bors-servo
Copy link

⌛ Testing commit cb12a31 with merge 1105757...

bors-servo pushed a commit that referenced this pull request Sep 21, 2018
codegen: Explicitly align unions if needed.

Also adds a test header I missed from the previous PR.

Fixes #1393
@bors-servo
Copy link

💔 Test failed - status-travis

@emilio
Copy link
Contributor Author

emilio commented Sep 21, 2018

Was a fluke.

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.

4 participants