-
Notifications
You must be signed in to change notification settings - Fork 13k
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
implement the chalk-engine traits #50937
Conversation
a2f0e10
to
5ef8cc9
Compare
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
c9e1a26
to
fd52186
Compare
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
fd52186
to
d5486da
Compare
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
} | ||
|
||
fn num_universes(_: &Canonical<'gcx, ty::ParamEnvAnd<'gcx, Goal<'gcx>>>) -> usize { | ||
0 // FIXME |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't it better to use an unimplemented!()
here?
d5486da
to
9dde1b2
Compare
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
src/librustc/ty/mod.rs
Outdated
impl<T> Eq for Slice<T> {} | ||
|
||
impl<T: PartialOrd> PartialOrd for Slice<T> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will conflict with #50930, this file can be reverted to master version.
9dde1b2
to
70f0030
Compare
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
70f0030
to
9ab03d9
Compare
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
9fc3dd1
to
b0a926f
Compare
b0a926f
to
8fd316f
Compare
@bors delegate=scalexm,leodasvacas -- I figure this is pretty chalk-focused, so they're best qualified to r+ |
📌 Commit 8fd316f has been approved by |
✌️ @scalexm,leodasvacas can now approve this pull request |
@bors r+ |
@scalexm: 🔑 Insufficient privileges: Not in reviewers |
@bors delegate=scalexm |
✌️ @scalexm can now approve this pull request |
@bors r=scalexm |
@bors r+ |
💡 This pull request was already approved, no need to approve it again.
|
📌 Commit 8fd316f has been approved by |
💡 This pull request was already approved, no need to approve it again.
|
📌 Commit 8fd316f has been approved by |
⌛ Testing commit 8fd316f with merge 670f4dcb166188e86423f6e914ce6d14e556b6f5... |
💔 Test failed - status-travis |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
travis failures look spurious. |
That error is caused by travis itself (it wasn't able to download a ruby gem it needs to deploy things). @bors retry |
implement the chalk-engine traits Preliminary implementation for the Chalk traits in rustc. Lots of `panic!()` placeholders to be filled in later. This is currently blocked on us landing rust-lang/chalk#131 in chalk and issuing a new release, which should occur later today. r? @scalexm cc @leodasvacas
☀️ Test successful - status-appveyor, status-travis |
Preliminary implementation for the Chalk traits in rustc. Lots of
panic!()
placeholders to be filled in later.This is currently blocked on us landing rust-lang/chalk#131 in chalk and issuing a new release, which should occur later today.
r? @scalexm
cc @leodasvacas