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 some codegen hacks #850

Merged
merged 1 commit into from Dec 11, 2016

Conversation

Projects
None yet
3 participants
@brson
Copy link
Contributor

brson commented Dec 10, 2016

These were to work around bugs in older compiler revisions.

r? @alexcrichton

Remove some codegen hacks
These were to work around bugs in older compiler revisions.
@alexcrichton

This comment has been minimized.

Copy link
Member

alexcrichton commented Dec 10, 2016

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Dec 10, 2016

📌 Commit e308c56 has been approved by alexcrichton

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Dec 10, 2016

⌛️ Testing commit e308c56 with merge 9d8d0bc...

bors added a commit that referenced this pull request Dec 10, 2016

Auto merge of #850 - brson:nohacks, r=alexcrichton
Remove some codegen hacks

These were to work around bugs in older compiler revisions.

r? @alexcrichton
@bors

This comment has been minimized.

Copy link
Contributor

bors commented Dec 11, 2016

💔 Test failed - status-appveyor

@alexcrichton

This comment has been minimized.

Copy link
Member

alexcrichton commented Dec 11, 2016

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Dec 11, 2016

⌛️ Testing commit e308c56 with merge 51aa295...

bors added a commit that referenced this pull request Dec 11, 2016

Auto merge of #850 - brson:nohacks, r=alexcrichton
Remove some codegen hacks

These were to work around bugs in older compiler revisions.

r? @alexcrichton
@bors

This comment has been minimized.

Copy link
Contributor

bors commented Dec 11, 2016

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 51aa295 to master...

@bors bors merged commit e308c56 into rust-lang:master Dec 11, 2016

3 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
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.