Skip to content

Fix regression from #7822 about exnrefs with nulls and unknowns#7835

Merged
kripken merged 2 commits intoWebAssembly:mainfrom
kripken:gufa.eh.null.type
Aug 15, 2025
Merged

Fix regression from #7822 about exnrefs with nulls and unknowns#7835
kripken merged 2 commits intoWebAssembly:mainfrom
kripken:gufa.eh.null.type

Conversation

@kripken
Copy link
Copy Markdown
Member

@kripken kripken commented Aug 15, 2025

RootLocation, added in #7822, was overly-ambitious in how much it
simplified. It turns out that there are situations where we need the same
type for both a null and an "unknown" of that type, and a single RootLocation
is therefore not enough (the specific situation the fuzzer found involves
exnref, but perhaps others are possible too).

To fix this, refactor back to two locations, NullLocation and TypeLocation,
where the latter is at least shared by exn and cont as a place with an
unknown value of a type.

@kripken kripken requested a review from tlively August 15, 2025 22:34
@kripken kripken merged commit 74ae98b into WebAssembly:main Aug 15, 2025
16 checks passed
@kripken kripken deleted the gufa.eh.null.type branch August 15, 2025 23:29
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