Skip to content

Commit 89cf437

Browse files
authored
Fix other.test_split_module_wasm64 test. NFC (#25531)
This should have been part of #25432 but it seems we had a gap in our wasm64 testing because the `other` test suite is run with the standard emsdk version of node (v18) which does not support wasm64.
1 parent 9a9beff commit 89cf437

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

.circleci/config.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -830,7 +830,7 @@ jobs:
830830
wasm64l.test_module_wasm_memory
831831
wasm64l.test_longjmp2_emscripten
832832
wasm64l.test_embind_val_basics_legacy
833-
other.test_failing_growth_wasm64"
833+
other.*_wasm64"
834834
- upload-test-results
835835
test-jsc:
836836
executor: linux-python

test/other/test_split_module.post.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,11 @@
1+
#preprocess
2+
13
function saveProfileData() {
24
var __write_profile = wasmExports['__write_profile'];
35
if (__write_profile) {
4-
var len = __write_profile(0, 0);
6+
var len = __write_profile({{{ to64('0') }}}, 0);
57
var offset = _malloc(len);
6-
var actualLen = __write_profile(offset, len);
8+
var actualLen = __write_profile({{{ to64('offset') }}}, len);
79
var profile_data = HEAPU8.subarray(offset, offset + len);
810
if (typeof writeFile !== 'undefined') {
911
console.log('using writeFile')

0 commit comments

Comments
 (0)