Skip to content

Conversation

kripken
Copy link
Member

@kripken kripken commented Aug 27, 2025

Many instructions operate on the contents of e.g. an i32 (binary ops,
unary ops, ifs and branches, etc.), but ref.eq (and maybe descriptor ops)
are the only things that test reference identity, so we may have been
underfuzzing that.

Might help #7857 (review)

@kripken kripken requested a review from tlively August 27, 2025 14:57
Nop : 7
Pop : 2
RefAs : 1
RefEq : 4
Copy link
Member

Choose a reason for hiding this comment

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

Ironic that the number of RefEq did not increase.

@kripken kripken merged commit 1785bc3 into WebAssembly:main Aug 28, 2025
15 of 16 checks passed
@kripken kripken deleted the fuzz.ref.eq.very branch August 28, 2025 16:40
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