Skip to content

Disable superseded Wasm benchmarks by default#178

Merged
danleh merged 1 commit into
WebKit:mainfrom
danleh:disable-gcc-loops-quicksort
Sep 10, 2025
Merged

Disable superseded Wasm benchmarks by default#178
danleh merged 1 commit into
WebKit:mainfrom
danleh:disable-gcc-loops-quicksort

Conversation

@danleh
Copy link
Copy Markdown
Contributor

@danleh danleh commented Sep 8, 2025

As discussed in last week's sync. Specifically gcc-loops and quicksort are very micro-benchmarky, and this reduces the proportion of Wasm workloads again a bit.

(Also moved all disabled/old Wasm workloads to the beginning of the Wasm list now. We can still decide to disable tsf-wasm later, since 8bitbench-wasm is also quite dispatch heavy and possibly a nice replacement.)

@eqrion @kmiller68 Let me know if I can merge this, then we can make some progress on the other open Wasm PRs.

As discussed in last week's sync. Specifically `gcc-loops` and `quicksort` are very micro-benchmarky, and this reduces the proportion of Wasm workloads again a bit.
@danleh danleh requested a review from kmiller68 September 8, 2025 13:57
@netlify
Copy link
Copy Markdown

netlify Bot commented Sep 8, 2025

Deploy Preview for webkit-jetstream-preview ready!

Name Link
🔨 Latest commit 32d881c
🔍 Latest deploy log https://app.netlify.com/projects/webkit-jetstream-preview/deploys/68bee0ca395bf70008acdf4d
😎 Deploy Preview https://deploy-preview-178--webkit-jetstream-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Contributor

@kmiller68 kmiller68 left a comment

Choose a reason for hiding this comment

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

LGTM

@danleh danleh merged commit ae41bb8 into WebKit:main Sep 10, 2025
10 checks passed
@danleh danleh deleted the disable-gcc-loops-quicksort branch September 16, 2025 09:55
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