Repository navigation
Conversation
c54ff2e to
a923b6f
Compare
|
@bors try @rust-timer queue |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
[PERF EXPERIMENT] Drop the previous-to-current dep-graph index mapping
This comment has been minimized.
This comment has been minimized.
|
Finished benchmarking commit (b81e23e): comparison URL. Overall result: ❌✅ regressions and improvements - please read:Benchmarking means the PR may be perf-sensitive. It's automatically marked not fit for rolling up. Overriding is possible but disadvised: it risks changing compiler perf. Next, please: If you can, justify the regressions found in this try perf run in writing along with @bors rollup=never rustc-perf Instruction countOur most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.
Max RSS (memory usage)Results (primary -0.9%, secondary 0.8%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary -3.1%, secondary -2.2%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis perf run didn't have relevant results for this metric. Bootstrap: 488.596s -> 489.56s (0.20%) |
|
☔ The latest upstream changes (presumably #165563) made this pull request unmergeable. Please resolve the merge conflicts by rebasing. |
Follow up of #164065.
Just testing perf for now. This PR has some cleanup because of the new invariant that
prev_index == new_indexin the depgraph which we want to introduce in #164065. This also allows us to stop collecting edges during the green marking walk, which should improve perf.