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

Store all generic arguments for method calls in AST/HIR #43115

Merged
merged 3 commits into from
Jul 9, 2017

Conversation

petrochenkov
Copy link
Contributor

The first part of #42492.
Landed separately to start the process of merging libsyntax changes breaking rustfmt, which is not easy these days.

@rust-highfive
Copy link
Collaborator

r? @pnkfelix

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

@petrochenkov
Copy link
Contributor Author

@bors r=eddyb
(#42492 (comment))

@bors
Copy link
Contributor

bors commented Jul 8, 2017

📌 Commit 31c8f7c has been approved by eddyb

@bors
Copy link
Contributor

bors commented Jul 8, 2017

⌛ Testing commit 31c8f7c4f93998e6090e1f86229ca263cad5519a with merge df78f1a76f512728aa51c4345ead1cac26d46a82...

@bors
Copy link
Contributor

bors commented Jul 8, 2017

💔 Test failed - status-appveyor

@petrochenkov
Copy link
Contributor Author

Looks spurious.
@bors retry

@bors
Copy link
Contributor

bors commented Jul 9, 2017

⌛ Testing commit 31c8f7c4f93998e6090e1f86229ca263cad5519a with merge 3934cc85da94a14bbf66f31b92b977f9b6956b42...

@bors
Copy link
Contributor

bors commented Jul 9, 2017

💔 Test failed - status-travis

@petrochenkov
Copy link
Contributor Author

RLS build fails on PowerPC with unresolved import `std::sync::atomic::AtomicU64` due to rust-lang/rls@7029815

@petrochenkov
Copy link
Contributor Author

@bors r=eddyb

@bors
Copy link
Contributor

bors commented Jul 9, 2017

📌 Commit 9ac79e4 has been approved by eddyb

@bors
Copy link
Contributor

bors commented Jul 9, 2017

⌛ Testing commit 9ac79e4 with merge d84693b...

bors added a commit that referenced this pull request Jul 9, 2017
Store all generic arguments for method calls in AST/HIR

The first part of #42492.
Landed separately to start the process of merging libsyntax changes breaking rustfmt, which is not easy these days.
@bors
Copy link
Contributor

bors commented Jul 9, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: eddyb
Pushing d84693b to master...

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

Successfully merging this pull request may close these issues.

5 participants