Avoid marshalling FFI_TYPE_LONGDOUBLE when WASM_BIGINT is not used

hoodmane · Jun 26, 2021 · 61bd5a3 · 61bd5a3
1 parent 139f8e1
commit 61bd5a3
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/wasm32/ffi.c b/src/wasm32/ffi.c
@@ -226,10 +226,12 @@ ffi_call, (ffi_cif * cif, ffi_fp fn, void *rvalue, void **avalue),
  SIG(sig += 'd');
  break;
  case FFI_TYPE_LONGDOUBLE:
- // emscripten doesn't define HEAPU64 by default
- const HEAPU64 = new BigInt64Array(HEAP8.buffer);
+#if WASM_BIGINT
  args.push(DEREF_U64(arg_ptr, 0));
  args.push(DEREF_U64(arg_ptr, 1));
+#else
+ throw new Error('longdouble marshalling nyi');
+#endif
  break;
  case FFI_TYPE_UINT8:
  args.push(HEAPU8[arg_ptr]);
@@ -479,7 +481,6 @@ ffi_prep_closure_loc_helper,
  }
  cur_ptr -= 4 * (sig.length - 1 - ret_by_arg);
  var args_ptr = cur_ptr;
- var HEAPU64 = new BigInt64Array(HEAP8.buffer);
  for (var sig_idx = 1 + ret_by_arg; sig_idx < sig.length; sig_idx++) {
  var jsargs_idx = sig_idx - 1;
  var cargs_idx = jsargs_idx - ret_by_arg;