WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
275989
Kotlin Compose demos (using WasmGC) no longer work in the latest Safari TP
https://bugs.webkit.org/show_bug.cgi?id=275989
Summary
Kotlin Compose demos (using WasmGC) no longer work in the latest Safari TP
bashorov
Reported
2024-06-28 06:00:50 PDT
To reproduce: 1. Run Safari TP with `__XPC_JSC_useWebAssemblyTypedFunctionReferences=1 __XPC_JSC_useWebAssemblyGC=1` 2. Open
https://zal.im/wasm/jetsnack/
or
https://zal.im/wasm/iv/
Expected: an application is working. Actual: a page is crashed.
Attachments
Add attachment
proposed patch, testcase, etc.
Ahmad Saleem
Comment 1
2024-06-28 06:49:03 PDT
@Bashorov - can you attach crash log to enable / help us to work forward?
bashorov
Comment 2
2024-06-28 06:58:17 PDT
(In reply to Ahmad Saleem from
comment #1
)
> @Bashorov - can you attach crash log to enable / help us to work forward?
Do you have an instruction on how/where I can find a crash log?
bashorov
Comment 3
2024-06-28 06:59:40 PDT
(In reply to Ahmad Saleem from
comment #1
)
> @Bashorov - can you attach crash log to enable / help us to work forward?
Alternatively, I can help to reproduce it. It should be easy, see the instruction.
Alexey Proskuryakov
Comment 4
2024-06-28 17:44:10 PDT
The crash log goes into ~/Library/Logs/DiagnosticReports. I can reproduce with STP 197. You said "no longer works", do you know when this last worked? Thread 10 Crashed:: Wasm Worklist Helper Thread 0 JavaScriptCore 0x10d35be4c JSC::Wasm::WasmCallingConvention::callInformationFor(JSC::Wasm::FunctionSignature const&, JSC::Wasm::CallRole) const + 116 1 JavaScriptCore 0x10e351158 JSC::Wasm::OMGIRGenerator::addCall(unsigned int, JSC::Wasm::TypeDefinition const&, WTF::Vector<JSC::B3::Variable*, 0ul, WTF::CrashOnOverflow, 16ul, WTF::FastMalloc>&, WTF::Vector<JSC::B3::Variable*, 8ul, WTF::CrashOnOverflow, 16ul, WTF::FastMalloc>&, JSC::CallLinkInfoBase::CallType) + 232 2 JavaScriptCore 0x10e3ab060 JSC::Wasm::FunctionParser<JSC::Wasm::OMGIRGenerator>::parseExpression() + 20440 3 JavaScriptCore 0x10e350b3c JSC::Wasm::FunctionParser<JSC::Wasm::OMGIRGenerator>::parse() + 8196 4 JavaScriptCore 0x10e35285c JSC::Wasm::OMGIRGenerator::addCall(unsigned int, JSC::Wasm::TypeDefinition const&, WTF::Vector<JSC::B3::Variable*, 0ul, WTF::CrashOnOverflow, 16ul, WTF::FastMalloc>&, WTF::Vector<JSC::B3::Variable*, 8ul, WTF::CrashOnOverflow, 16ul, WTF::FastMalloc>&, JSC::CallLinkInfoBase::CallType) + 6124 5 JavaScriptCore 0x10e3ab060 JSC::Wasm::FunctionParser<JSC::Wasm::OMGIRGenerator>::parseExpression() + 20440 6 JavaScriptCore 0x10e350b3c JSC::Wasm::FunctionParser<JSC::Wasm::OMGIRGenerator>::parse() + 8196 7 JavaScriptCore 0x10e358c40 JSC::Wasm::parseAndCompileOMG(JSC::Wasm::CompilationContext&, JSC::Wasm::OptimizingJITCallee&, JSC::Wasm::FunctionData const&, JSC::Wasm::TypeDefinition const&, WTF::Vector<JSC::Wasm::UnlinkedWasmToWasmCall, 0ul, WTF::CrashOnOverflow, 16ul, WTF::FastMalloc>&, JSC::Wasm::CalleeGroup&, JSC::Wasm::ModuleInformation const&, JSC::MemoryMode, JSC::Wasm::CompilationMode, unsigned int, std::__1::optional<bool>, unsigned int, JSC::Wasm::TierUpCount*) + 4256 8 JavaScriptCore 0x10e391ff8 JSC::Wasm::OMGPlan::work(JSC::Wasm::Plan::CompilationEffort) + 324
bashorov
Comment 5
2024-06-29 10:33:24 PDT
I think it worked about a month ago (in May) in Safari TP; I don't know the specific version, but probably it was 196. Examples are working with WebKit
275934@main
.
Radar WebKit Bug Importer
Comment 6
2024-07-05 06:01:40 PDT
<
rdar://problem/131166278
>
Justin Michaud
Comment 7
2024-07-05 09:08:27 PDT
I can reproduce this. It looks like it shouldn't be too hard to fix
David Degazio
Comment 8
2024-07-31 17:45:35 PDT
Pull request:
https://github.com/WebKit/WebKit/pull/31568
EWS
Comment 9
2024-08-02 10:54:24 PDT
Committed
281767@main
(08d5b5ec570f): <
https://commits.webkit.org/281767@main
> Reviewed commits have been landed. Closing PR #31568 and removing active labels.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug