rustdoc-search: simplify the checkTypes fast path#117955
rustdoc-search: simplify the checkTypes fast path#117955bors merged 1 commit intorust-lang:masterfrom
Conversation
This reduces code size while still matching the common case for plain, concrete types.
|
Some changes occurred in HTML/CSS/JS. cc @GuillaumeGomez, @jsha |
GuillaumeGomez
left a comment
There was a problem hiding this comment.
Less code for same output and better performance? That's awesome! Thanks a lot for this improvement!
|
@bors r+ rollup |
|
☀️ Test successful - checks-actions |
|
Finished benchmarking commit (cd05ffa): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)ResultsThis is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
CyclesResultsThis is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 675.809s -> 675.207s (-0.09%) |
This reduces code size while still matching the common case for plain, concrete types. This should not introduce any user-visible behavior changes.
The perf impact of the change was evaluated using https://gitlab.com/notriddle/rustdoc-js-profile. The output of the tool can be found here: http://notriddle.com/rustdoc-html-demo-5/profile-1/index.html
r? @GuillaumeGomez