New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Wasm-GC] Fix null checks for struct operations #9091
[Wasm-GC] Fix null checks for struct operations #9091
Conversation
143ade5
to
2f998a6
Compare
EWS run on previous version of this PR (hash 2f998a6)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r=me
2f998a6
to
ca97dce
Compare
EWS run on previous version of this PR (hash ca97dce)
|
ca97dce
to
21ed12c
Compare
EWS run on current version of this PR (hash 21ed12c)
|
https://bugs.webkit.org/show_bug.cgi?id=251133 Reviewed by Justin Michaud. Adds in missing null checks for struct instructions. * JSTests/wasm/gc/structs.js: * Source/JavaScriptCore/wasm/WasmAirIRGeneratorBase.h: (JSC::Wasm::ExpressionType>::addStructGet): (JSC::Wasm::ExpressionType>::addStructSet): * Source/JavaScriptCore/wasm/WasmB3IRGenerator.cpp: (JSC::Wasm::B3IRGenerator::addStructGet): (JSC::Wasm::B3IRGenerator::addStructSet): * Source/JavaScriptCore/wasm/WasmExceptionType.h: (JSC::Wasm::isTypeErrorExceptionType): * Source/JavaScriptCore/wasm/WasmSlowPaths.cpp: (JSC::LLInt::WASM_SLOW_PATH_DECL): Canonical link: https://commits.webkit.org/259449@main
21ed12c
to
a5a8768
Compare
Committed 259449@main (a5a8768): https://commits.webkit.org/259449@main Reviewed commits have been landed. Closing PR #9091 and removing active labels. |
a5a8768
21ed12c