Skip to content

Fix an ICE in BuildReducedGraphVisitor::visit_trait_item. #36951

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

Merged
merged 1 commit into from
Oct 4, 2016

Conversation

nnethercote
Copy link
Contributor

@nnethercote nnethercote commented Oct 4, 2016

This ICE occurs in the futures-rs-test-all benchmark in rustc-benchmarks (fixes #36950).

@rust-highfive
Copy link
Contributor

r? @arielb1

(rust_highfive has picked a reviewer for you, use r? to override)

This ICE occurs in the futures-rs-test-all benchmark in
rustc-benchmarks.
@eddyb
Copy link
Member

eddyb commented Oct 4, 2016

@bors r+

@bors
Copy link
Collaborator

bors commented Oct 4, 2016

📌 Commit 7857903 has been approved by eddyb

@nnethercote
Copy link
Contributor Author

Thanks to @eddyb for telling me how to fix this!

Manishearth added a commit to Manishearth/rust that referenced this pull request Oct 4, 2016
Fix an ICE in BuildReducedGraphVisitor::visit_trait_item.

This ICE occurs in the futures-rs-test-all benchmark in rustc-benchmarks (fixes rust-lang#36950).
bors added a commit that referenced this pull request Oct 4, 2016
Rollup of 12 pull requests

- Successful merges: #36798, #36878, #36902, #36903, #36908, #36916, #36917, #36921, #36928, #36938, #36941, #36951
- Failed merges:
@bors bors merged commit 7857903 into rust-lang:master Oct 4, 2016
@nnethercote nnethercote deleted the fix-ICE branch October 4, 2016 21:51
@jseyfried
Copy link
Contributor

@nnethercote Thanks! That ICE was my fault (introduced in #3660).
Also, this PR fixes #36943.

# 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.

ICE building futures-rs-all-test from rustc-benchmarks
6 participants