-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
internal: Some method resolution cleanups #16749
Conversation
fd93e97
to
516330c
Compare
ad25dae
to
b05a197
Compare
@@ -438,6 +438,7 @@ impl<'a> InferenceTable<'a> { | |||
where | |||
T: HasInterner<Interner = Interner> + TypeFoldable<Interner>, | |||
{ | |||
// TODO check this vec here |
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.
what sort of check would happen here?
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.
Oh that was a note to myself to check if we can cache the allocation for that vec here
When I tried this out yesterday, this yield a ~10ms improvement on completions! |
Hopefully the saves don't come from fcc246e as that is likely to be invalid |
Dropped fcc246e for now as it fails the test I've added after. |
b05a197
to
c679482
Compare
@bors r+ |
☀️ Test successful - checks-actions |
fix: Fix method resolution snapshotting receiver_ty too early Accidental change in #16749 presumably caused some type mismatches in webrender
No description provided.