Skip to content

Add webrender 2022 again #1252

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

Merged
merged 2 commits into from
Mar 28, 2022
Merged

Conversation

nnethercote
Copy link
Contributor

webrender-2022 was originally added in #1236, then backed out in #1237. Let's try again.

Here's how I obtained it. It required a little dance because the repo
name `webrender` is the same as the existing benchmark `webrender`.
```
mv collector/benchmarks/webrender collector/benchmarks/webrender-old
target/release/collector download -c primary git https://github.com/servo/webrender
mv collector/benchmarks/webrender collector/benchmarks/webrender-2022
mv collector/benchmarks/webrender-old collector/benchmarks/webrender
git add collector/benchmarks/webrender-2022
```
Here's the overall time to `full` compile everything, webrender vs
webrender-2022:
```
         old    new
- check  15.8s  21.9s
- debug  19.5s  28.4s
- opt    25.0s  38.9s
```
Roughly 1.5x longer.

And here's the time to `full` compile just the final crate:
```
         old    new
- check  1.74s  4.28s
- debug  4.70s  9.92s
- opt    9.58s  19.14s
```
Roughly 2x longer.
@nnethercote nnethercote requested a review from rylev March 28, 2022 06:22
Copy link
Member

@rylev rylev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI: I believe it was backed out in #1242 when we were unsure what was causing the issues with the status page.

@rylev rylev merged commit dcc9b10 into rust-lang:master Mar 28, 2022
@nnethercote nnethercote deleted the add-webrender-2022-b branch March 28, 2022 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants