Skip to content

Commit ac3758d

Browse files
authored
Fix other.test_closure_webgpu_wasm64 (#26955)
This was broken by #26954 but not covered by CI because we run the `other` testsuite with the emsdk version of node that doesn't support wasm64. Without this change the wasm64 variant complains because it tried to case the return value of this function to a BigInt (pointer) before returning to Wasm.
1 parent 4e0e1bc commit ac3758d

3 files changed

Lines changed: 4 additions & 2 deletions

File tree

.circleci/config.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1081,6 +1081,7 @@ jobs:
10811081
other.test_min_node_version
10821082
other.test_node_emscripten_num_logical_cores
10831083
other.test_js_base64_api
1084+
other.test_closure_webgpu_wasm64
10841085
core2.test_hello_world
10851086
core2.test_pthread_create
10861087
core2.test_i64_invoke_bigint

src/lib/libpromise.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -274,6 +274,7 @@ addToLibrary({
274274
},
275275
emscripten_promise_await_unchecked: (id) => {
276276
abort('emscripten_promise_await_unchecked is only available with ASYNCIFY');
277+
return 0;
277278
},
278279
#endif
279280
});

test/codesize/test_codesize_hello_dylink_all.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
2-
"a.out.js": 268222,
2+
"a.out.js": 268231,
33
"a.out.nodebug.wasm": 587563,
4-
"total": 855785,
4+
"total": 855794,
55
"sent": [
66
"IMG_Init",
77
"IMG_Load",

0 commit comments

Comments
 (0)