Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upresolve: Improve performance #33064
Conversation
rust-highfive
assigned
eddyb
Apr 18, 2016
jseyfried
referenced this pull request
Apr 18, 2016
Closed
resolution time has *quadrupled* between 235d77457 and 6fa61b810 #33061
jseyfried
force-pushed the
jseyfried:improve_resolve_performance
branch
from
a9660f6
to
6ae8027
Apr 18, 2016
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
@bors r+ |
This comment has been minimized.
This comment has been minimized.
|
|
This comment has been minimized.
This comment has been minimized.
bors
added a commit
that referenced
this pull request
Apr 18, 2016
This comment has been minimized.
This comment has been minimized.
|
@eddyb thanks! The regression is currently in beta -- do you think we should beta-nominate this? |
eddyb
added
the
beta-nominated
label
Apr 18, 2016
This comment has been minimized.
This comment has been minimized.
bors
merged commit 6ae8027
into
rust-lang:master
Apr 18, 2016
This comment has been minimized.
This comment has been minimized.
|
Ordinarily we wouldn't backport a pure performance change, but approved for beta because:
|
nikomatsakis
added
the
beta-accepted
label
Apr 21, 2016
brson
added
relnotes
and removed
beta-nominated
labels
May 4, 2016
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
jseyfried commentedApr 18, 2016
•
edited
This fixes #33061 by speeding up searchs through all traits in scope, a bottleneck in
resolve.According to my measurements, this PR improves resolution performance by a factor of 3.4x on
librustc, which almost completely reverses the 3.75x performance regression reported in #33061.r? @eddyb