Skip to content

[scripts] Update name of experimental Wasm V8 flags - #9134

Open
Liedtke wants to merge 1 commit into
WebAssembly:mainfrom
Liedtke:03_rename_d8_flags
Open

Liedtke wants to merge 1 commit into
WebAssembly:mainfrom
Liedtke:03_rename_d8_flags

Conversation

@Liedtke

@Liedtke Liedtke commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

These flags got renamed in https://crrev.com/c/8426472.

Note that this is just the code change, I don't know if there is any pipeline in place to automatically update the ClusterFuzz fuzzer.

@Liedtke
Liedtke requested a review from a team as a code owner September 22, 2026 10:39
@Liedtke
Liedtke requested review from stevenfontanella and removed request for a team September 22, 2026 10:39
@stevenfontanella

Copy link
Copy Markdown
Member

@Liedtke can you run fuzz_opt.py as a sanity check?

Also I guess this would break the fuzzer with earlier versions of V8 that don't have the renamed flags. Is that fine @kripken? I get the impression that we don't care that much about backward compatibility with earlier versions of V8 for fuzzing.

@kripken

kripken commented Sep 22, 2026

Copy link
Copy Markdown
Member

Yes, we don't care about older V8 - we just expect people to update their V8.

@stevenfontanella

Copy link
Copy Markdown
Member

LGTM if scripts/fuzz_opt.py runs successfully for a few iterations.

This branch has not been deployed

No deployments
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.

3 participants