khagankhan requested fitzgen for a review on PR #14196.
khagankhan requested wasmtime-fuzz-reviewers for a review on PR #14196.
khagankhan opened PR #14196 from khagankhan:array-ops to bytecodealliance:main:
This PR adds new GC ops to the existing setup.
- `array.fill'
array.copyarray.new_elemarray.new_fixed+cc @fitzgen
khagankhan edited PR #14196.
khagankhan edited PR #14196.
github-actions[bot] added the label fuzzing on PR #14196.
github-actions[bot] commented on PR #14196:
Subscribe to Label Action
cc @fitzgen
<details>
This issue or pull request has been labeled: "fuzzing"Thus the following users have been cc'd because of the following labels:
- fitzgen: fuzzing
To subscribe or unsubscribe from this label, edit the <code>.github/subscribe-to-label.json</code> configuration file.
Learn more.
</details>
:memo: fitzgen submitted PR review:
Generally looks good but confused about one part bel
:speech_balloon: fitzgen created PR review comment:
I don't understand the purpose of this change?
:memo: fitzgen submitted PR review:
Generally looks good but confused about one part below
:memo: khagankhan submitted PR review.
:speech_balloon: khagankhan created PR review comment:
This second bank of locals is for
array.copy.Because both operands can be the same concrete array type andarray.copytraps on Null it needs two locals for src and dst when checking. IP.S. It also does it for struct locals too but I did not want to "separate" arrays from structs atm.
khagankhan updated PR #14196.
khagankhan updated PR #14196.
:thumbs_up: fitzgen submitted PR review.
:memo: fitzgen submitted PR review.
:speech_balloon: fitzgen created PR review comment:
Can you split this method up into some smaller helpers and add comments and such in a follow up PR? It's pretty big at this point and it isn't immediately clear to readers what is happening.
fitzgen added PR #14196 [gc_fuzz]: Add new GC ops to the fuzzer setup to the merge queue.
:memo: khagankhan submitted PR review.
:speech_balloon: khagankhan created PR review comment:
Yes! o7
:check: fitzgen merged PR #14196.
fitzgen removed PR #14196 [gc_fuzz]: Add new GC ops to the fuzzer setup from the merge queue.
Last updated: Sep 20 2026 at 19:05 UTC