binaryen icon indicating copy to clipboard operation
binaryen copied to clipboard

wasm2js/I64ToI32Lowering fails on 64-bit atomic stores (I think) test_atomic_wait64_notify

Open sbc100 opened this issue 1 year ago • 6 comments

I'm currently landing a new test test_atomic_wait64_notify which is failing under wasm2js:

test_atomic_wait64_notify (test_core.wasm2js1.test_atomic_wait64_notify) ... Clearing existing test directory
wasm2js: /usr/local/google/home/sbc/dev/wasm/binaryen/src/passes/I64ToI32Lowering.cpp:423: void wasm::I64ToI32Lowering::visitStore(wasm::Store *): Assertion `!curr->isAtomic && "atomic store not implemented"' failed.
emcc: error: '/usr/local/google/home/sbc/dev/wasm/binaryen-out/bin/wasm2js --emscripten -O test_atomic_wait64_notify.wasm --mvp-features --enable-threads --enable-bulk-memory --enable-mutable-globals --enable-sign-ext' failed (received SIGABRT (-6))
None
None
FAIL

sbc100 avatar Oct 03 '23 19:10 sbc100

See https://github.com/emscripten-core/emscripten/pull/20381

sbc100 avatar Oct 03 '23 19:10 sbc100