[EXPERIMENT] Use SmallVec in GoalStalledOn - #161270
Conversation
It makes `GoalStalledOn` bigger, reversing rust-lang#160005, but that's not a problem any more because rust-lang#160479 massively reduced the number of `GoalStalledOn` copy operations, and inline operations are cheaper.
|
@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.
[EXPERIMENT] Use `SmallVec` in `GoalStalledOn`
This comment has been minimized.
This comment has been minimized.
|
Finished benchmarking commit (96a3828): 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 1.9%, secondary 2.3%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (secondary -6.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: 458.344s -> 465.594s (1.58%) |
|
A tiny icounts regression, some possible cycles improvements. @jdonszelmann, are you still planning that alternative ugly-but-effective enum representation for |
It makes
GoalStalledOnbigger, reversing #160005, but that's not a problem any more because #160479 massively reduced the number ofGoalStalledOncopy operations, and inline operations are cheaper.