koffi 2.15.1 → 2.16.0-beta.1

package/src/koffi/src/abi_arm64.cc

@@ -52,8 +52,6 @@ extern "C" napi_value CallSwitchStack(Napi::Function *func, size_t argc, napi_va
                                       uint8_t *old_sp, Span<uint8_t> *new_stack,
                                       napi_value (*call)(Napi::Function *func, size_t argc, napi_value *argv));
 
-#include "trampolines/prototypes.inc"
-
 static HfaInfo IsHFA(const TypeInfo *type)
 {
     bool float32 = false;
@@ -249,22 +247,45 @@ bool AnalyseFunction(Napi::Env, InstanceData *, FunctionInfo *func)
 
 bool CallData::Prepare(const FunctionInfo *func, const Napi::CallbackInfo &info)
 {
-    uint64_t *args_ptr = nullptr;
-    uint64_t *gpr_ptr = nullptr;
-    uint64_t *vec_ptr = nullptr;
+    uint64_t *gpr_ptr = AllocStack<uint64_t>(17 * 8 + func->args_size);
+    uint64_t *vec_ptr = gpr_ptr + 9;
+    uint64_t *args_ptr = gpr_ptr + 17;
 
-    // Return through registers unless it's too big
-    if (!AllocStack(func->args_size, 16, &args_ptr)) [[unlikely]]
-        return false;
-    if (!AllocStack(8 * 8, 8, &vec_ptr)) [[unlikely]]
-        return false;
-    if (!AllocStack(9 * 8, 8, &gpr_ptr)) [[unlikely]]
+    if (!gpr_ptr) [[unlikely]]
         return false;
     if (func->ret.use_memory) {
         return_ptr = AllocHeap(func->ret.type->size, 16);
         gpr_ptr[8] = (uint64_t)return_ptr;
     }
 
+    Size i = -1;
+
+#if defined(__GNUC__) || defined(__clang__)
+    static const void *const DispatchTable[] = {
+        #define PRIMITIVE(Name) && Name,
+        #include "primitives.inc"
+    };
+
+    #define LOOP
+    #define CASE(Primitive) \
+        do { \
+            PrimitiveKind next = func->primitives[++i]; \
+            goto *DispatchTable[(int)next]; \
+        } while (false); \
+        Primitive:
+    #define OR(Primitive) \
+        Primitive:
+#else
+    #define LOOP \
+        while (++i < func->parameters.len) \
+            switch (func->primitives[i])
+    #define CASE(Primitive) \
+        break; \
+        case PrimitiveKind::Primitive:
+    #define OR(Primitive) \
+        case PrimitiveKind::Primitive:
+#endif
+
 #if defined(_M_ARM64EC)
     if (func->variadic) {
         gpr_ptr[4] = (uint64_t)args_ptr;
@@ -280,13 +301,15 @@ bool CallData::Prepare(const FunctionInfo *func, const Napi::CallbackInfo &info)
 #if defined(__APPLE__)
     #define PUSH_INTEGER(CType) \
         do { \
-            if (!value.IsNumber() && !value.IsBigInt()) [[unlikely]] { \
+            const ParameterInfo &param = func->parameters[i]; \
+            Napi::Value value = info[param.offset]; \
+             \
+            CType v; \
+            if (!TryNumber(value, &v)) [[unlikely]] { \
                 ThrowError<Napi::TypeError>(env, "Unexpected %1 value, expected number", GetValueType(instance, value)); \
                 return false; \
             } \
              \
-            CType v = GetNumber<CType>(value); \
-             \
             if (param.gpr_count) [[likely]] { \
                 *(gpr_ptr++) = (uint64_t)v; \
             } else { \
@@ -297,13 +320,15 @@ bool CallData::Prepare(const FunctionInfo *func, const Napi::CallbackInfo &info)
         } while (false)
     #define PUSH_INTEGER_SWAP(CType) \
         do { \
-            if (!value.IsNumber() && !value.IsBigInt()) [[unlikely]] { \
+            const ParameterInfo &param = func->parameters[i]; \
+            Napi::Value value = info[param.offset]; \
+             \
+            CType v; \
+            if (!TryNumber(value, &v)) [[unlikely]] { \
                 ThrowError<Napi::TypeError>(env, "Unexpected %1 value, expected number", GetValueType(instance, value)); \
                 return false; \
             } \
              \
-            CType v = GetNumber<CType>(value); \
-             \
             if (param.gpr_count) [[likely]] { \
                 *(gpr_ptr++) = (uint64_t)ReverseBytes(v); \
             } else { \
@@ -315,227 +340,259 @@ bool CallData::Prepare(const FunctionInfo *func, const Napi::CallbackInfo &info)
 #else
     #define PUSH_INTEGER(CType) \
         do { \
-            if (!value.IsNumber() && !value.IsBigInt()) [[unlikely]] { \
+            const ParameterInfo &param = func->parameters[i]; \
+            Napi::Value value = info[param.offset]; \
+             \
+            CType v; \
+            if (!TryNumber(value, &v)) [[unlikely]] { \
                 ThrowError<Napi::TypeError>(env, "Unexpected %1 value, expected number", GetValueType(instance, value)); \
                 return false; \
             } \
              \
-            CType v = GetNumber<CType>(value); \
             *((param.gpr_count ? gpr_ptr : args_ptr)++) = (uint64_t)v; \
         } while (false)
     #define PUSH_INTEGER_SWAP(CType) \
         do { \
-            if (!value.IsNumber() && !value.IsBigInt()) [[unlikely]] { \
+            const ParameterInfo &param = func->parameters[i]; \
+            Napi::Value value = info[param.offset]; \
+             \
+            CType v; \
+            if (!TryNumber(value, &v)) [[unlikely]] { \
                 ThrowError<Napi::TypeError>(env, "Unexpected %1 value, expected number", GetValueType(instance, value)); \
                 return false; \
             } \
              \
-            CType v = GetNumber<CType>(value); \
             *((param.gpr_count ? gpr_ptr : args_ptr)++) = (uint64_t)ReverseBytes(v); \
         } while (false)
 #endif
 
     // Push arguments
-    for (Size i = 0; i < func->parameters.len; i++) {
-        const ParameterInfo &param = func->parameters[i];
-        K_ASSERT(param.directions >= 1 && param.directions <= 3);
+    LOOP {
+        CASE(Void) { K_UNREACHABLE(); };
 
-        Napi::Value value = info[param.offset];
-
-        switch (param.type->primitive) {
-            case PrimitiveKind::Void: { K_UNREACHABLE(); } break;
-
-            case PrimitiveKind::Bool: {
-                if (!value.IsBoolean()) [[unlikely]] {
-                    ThrowError<Napi::TypeError>(env, "Unexpected %1 value, expected boolean", GetValueType(instance, value));
-                    return false;
-                }
+        CASE(Bool) {
+            const ParameterInfo &param = func->parameters[i];
+            Napi::Value value = info[param.offset];
 
-                bool b = value.As<Napi::Boolean>();
+            bool b;
+            if (napi_get_value_bool(env, value, &b) != napi_ok) [[unlikely]] {
+                ThrowError<Napi::TypeError>(env, "Unexpected %1 value, expected boolean", GetValueType(instance, value));
+                return false;
+            }
 
 #if defined(__APPLE__)
-                if (param.gpr_count) [[likely]] {
-                    *(gpr_ptr++) = (uint64_t)b;
-                } else {
-                    *(uint8_t *)args_ptr = b;
-                    args_ptr = (uint64_t *)((uint8_t *)args_ptr + 1);
-                }
+            if (param.gpr_count) [[likely]] {
+                *(gpr_ptr++) = (uint64_t)b;
+            } else {
+                *(uint8_t *)args_ptr = b;
+                args_ptr = (uint64_t *)((uint8_t *)args_ptr + 1);
+            }
 #else
-                *((param.gpr_count ? gpr_ptr : args_ptr)++) = (uint64_t)b;
+            *((param.gpr_count ? gpr_ptr : args_ptr)++) = (uint64_t)b;
 #endif
-            } break;
-            case PrimitiveKind::Int8: { PUSH_INTEGER(int8_t); } break;
-            case PrimitiveKind::UInt8: { PUSH_INTEGER(uint8_t); } break;
-            case PrimitiveKind::Int16: { PUSH_INTEGER(int16_t); } break;
-            case PrimitiveKind::Int16S: { PUSH_INTEGER_SWAP(int16_t); } break;
-            case PrimitiveKind::UInt16: { PUSH_INTEGER(uint16_t); } break;
-            case PrimitiveKind::UInt16S: { PUSH_INTEGER_SWAP(uint16_t); } break;
-            case PrimitiveKind::Int32: { PUSH_INTEGER(int32_t); } break;
-            case PrimitiveKind::Int32S: { PUSH_INTEGER_SWAP(int32_t); } break;
-            case PrimitiveKind::UInt32: { PUSH_INTEGER(uint32_t); } break;
-            case PrimitiveKind::UInt32S: { PUSH_INTEGER_SWAP(uint32_t); } break;
-            case PrimitiveKind::Int64: { PUSH_INTEGER(int64_t); } break;
-            case PrimitiveKind::Int64S: { PUSH_INTEGER_SWAP(int64_t); } break;
-            case PrimitiveKind::UInt64: { PUSH_INTEGER(uint64_t); } break;
-            case PrimitiveKind::UInt64S: { PUSH_INTEGER_SWAP(uint64_t); } break;
-            case PrimitiveKind::String: {
-                const char *str;
-                if (!PushString(value, param.directions, &str)) [[unlikely]]
-                    return false;
+        };
+
+        CASE(Int8) { PUSH_INTEGER(int8_t); };
+        CASE(UInt8) { PUSH_INTEGER(uint8_t); };
+        CASE(Int16) { PUSH_INTEGER(int16_t); };
+        CASE(Int16S) { PUSH_INTEGER_SWAP(int16_t); };
+        CASE(UInt16) { PUSH_INTEGER(uint16_t); };
+        CASE(UInt16S) { PUSH_INTEGER_SWAP(uint16_t); };
+        CASE(Int32) { PUSH_INTEGER(int32_t); };
+        CASE(Int32S) { PUSH_INTEGER_SWAP(int32_t); };
+        CASE(UInt32) { PUSH_INTEGER(uint32_t); };
+        CASE(UInt32S) { PUSH_INTEGER_SWAP(uint32_t); };
+        CASE(Int64) { PUSH_INTEGER(int64_t); };
+        CASE(Int64S) { PUSH_INTEGER_SWAP(int64_t); };
+        CASE(UInt64) { PUSH_INTEGER(uint64_t); };
+        CASE(UInt64S) { PUSH_INTEGER_SWAP(uint64_t); };
+
+        CASE(String) {
+            const ParameterInfo &param = func->parameters[i];
+            Napi::Value value = info[param.offset];
+
+            const char *str;
+            if (!PushString(value, param.directions, &str)) [[unlikely]]
+                return false;
 
 #if defined(__APPLE__)
-                args_ptr = param.gpr_count ? args_ptr : AlignUp(args_ptr, 8);
+            args_ptr = param.gpr_count ? args_ptr : AlignUp(args_ptr, 8);
 #endif
-                *(const char **)((param.gpr_count ? gpr_ptr : args_ptr)++) = str;
-            } break;
-            case PrimitiveKind::String16: {
-                const char16_t *str16;
-                if (!PushString16(value, param.directions, &str16)) [[unlikely]]
-                    return false;
+            *(const char **)((param.gpr_count ? gpr_ptr : args_ptr)++) = str;
+        };
+        CASE(String16) {
+            const ParameterInfo &param = func->parameters[i];
+            Napi::Value value = info[param.offset];
+
+            const char16_t *str16;
+            if (!PushString16(value, param.directions, &str16)) [[unlikely]]
+                return false;
 
 #if defined(__APPLE__)
-                args_ptr = param.gpr_count ? args_ptr : AlignUp(args_ptr, 8);
+            args_ptr = param.gpr_count ? args_ptr : AlignUp(args_ptr, 8);
 #endif
-                *(const char16_t **)((param.gpr_count ? gpr_ptr : args_ptr)++) = str16;
-            } break;
-            case PrimitiveKind::String32: {
-                const char32_t *str32;
-                if (!PushString32(value, param.directions, &str32)) [[unlikely]]
-                    return false;
+            *(const char16_t **)((param.gpr_count ? gpr_ptr : args_ptr)++) = str16;
+        };
+        CASE(String32) {
+            const ParameterInfo &param = func->parameters[i];
+            Napi::Value value = info[param.offset];
+
+            const char32_t *str32;
+            if (!PushString32(value, param.directions, &str32)) [[unlikely]]
+                return false;
 
 #if defined(__APPLE__)
-                args_ptr = param.gpr_count ? args_ptr : AlignUp(args_ptr, 8);
+            args_ptr = param.gpr_count ? args_ptr : AlignUp(args_ptr, 8);
 #endif
-                *(const char32_t **)((param.gpr_count ? gpr_ptr : args_ptr)++) = str32;
-            } break;
-            case PrimitiveKind::Pointer: {
-                void *ptr;
-                if (!PushPointer(value, param.type, param.directions, &ptr)) [[unlikely]]
-                    return false;
+            *(const char32_t **)((param.gpr_count ? gpr_ptr : args_ptr)++) = str32;
+        };
+
+        CASE(Pointer) {
+            const ParameterInfo &param = func->parameters[i];
+            Napi::Value value = info[param.offset];
+
+            void *ptr;
+            if (!PushPointer(value, param.type, param.directions, &ptr)) [[unlikely]]
+                return false;
 
 #if defined(__APPLE__)
-                args_ptr = param.gpr_count ? args_ptr : AlignUp(args_ptr, 8);
+            args_ptr = param.gpr_count ? args_ptr : AlignUp(args_ptr, 8);
 #endif
-                *(void **)((param.gpr_count ? gpr_ptr : args_ptr)++) = ptr;
-            } break;
-            case PrimitiveKind::Record:
-            case PrimitiveKind::Union: {
-                if (!IsObject(value)) [[unlikely]] {
-                    ThrowError<Napi::TypeError>(env, "Unexpected %1 value, expected object", GetValueType(instance, value));
-                    return false;
-                }
+            *(void **)((param.gpr_count ? gpr_ptr : args_ptr)++) = ptr;
+        };
 
-                Napi::Object obj = value.As<Napi::Object>();
+        CASE(Record) OR(Union) {
+            const ParameterInfo &param = func->parameters[i];
+            Napi::Value value = info[param.offset];
 
-                if (param.vec_count) { // HFA
-                    uint8_t *ptr = (uint8_t *)vec_ptr;
+            if (!IsObject(value)) [[unlikely]] {
+                ThrowError<Napi::TypeError>(env, "Unexpected %1 value, expected object", GetValueType(instance, value));
+                return false;
+            }
 
-                    if (!PushObject(obj, param.type, ptr))
-                        return false;
-                    ExpandFloats(ptr, param.vec_count, param.vec_bytes);
+            Napi::Object obj = value.As<Napi::Object>();
 
-                    vec_ptr += param.vec_count;
-                } else if (!param.use_memory) {
-                    if (param.gpr_count) {
-                        K_ASSERT(param.type->align <= 8);
+            if (param.vec_count) { // HFA
+                uint8_t *ptr = (uint8_t *)vec_ptr;
 
-                        if (!PushObject(obj, param.type, (uint8_t *)gpr_ptr))
-                            return false;
-                        gpr_ptr += param.gpr_count;
-                    } else if (param.type->size) {
+                if (!PushObject(obj, param.type, ptr))
+                    return false;
+                ExpandFloats(ptr, param.vec_count, param.vec_bytes);
+
+                vec_ptr += param.vec_count;
+            } else if (!param.use_memory) {
+                if (param.gpr_count) {
+                    K_ASSERT(param.type->align <= 8);
+
+                    if (!PushObject(obj, param.type, (uint8_t *)gpr_ptr))
+                        return false;
+                    gpr_ptr += param.gpr_count;
+                } else if (param.type->size) {
 #if defined(__APPLE__)
-                        args_ptr = AlignUp(args_ptr, 8);
+                    args_ptr = AlignUp(args_ptr, 8);
 #endif
-                        if (!PushObject(obj, param.type, (uint8_t *)args_ptr))
-                            return false;
-                        args_ptr += (param.type->size + 7) / 8;
-                    }
-                } else {
-                    uint8_t *ptr = AllocHeap(param.type->size, 16);
+                    if (!PushObject(obj, param.type, (uint8_t *)args_ptr))
+                        return false;
+                    args_ptr += (param.type->size + 7) / 8;
+                }
+            } else {
+                uint8_t *ptr = AllocHeap(param.type->size, 16);
 
-                    if (param.gpr_count) {
-                        K_ASSERT(param.gpr_count == 1);
-                        K_ASSERT(param.vec_count == 0);
+                if (param.gpr_count) {
+                    K_ASSERT(param.gpr_count == 1);
+                    K_ASSERT(param.vec_count == 0);
 
-                        *(uint8_t **)(gpr_ptr++) = ptr;
-                    } else {
+                    *(uint8_t **)(gpr_ptr++) = ptr;
+                } else {
 #if defined(__APPLE__)
-                        args_ptr = AlignUp(args_ptr, 8);
+                    args_ptr = AlignUp(args_ptr, 8);
 #endif
-                        *(uint8_t **)(args_ptr++) = ptr;
-                    }
-
-                    if (!PushObject(obj, param.type, ptr))
-                        return false;
+                    *(uint8_t **)(args_ptr++) = ptr;
                 }
-            } break;
-            case PrimitiveKind::Array: { K_UNREACHABLE(); } break;
-            case PrimitiveKind::Float32: {
-                if (!value.IsNumber() && !value.IsBigInt()) [[unlikely]] {
-                    ThrowError<Napi::TypeError>(env, "Unexpected %1 value, expected number", GetValueType(instance, value));
+
+                if (!PushObject(obj, param.type, ptr))
                     return false;
-                }
+            }
+        };
+        CASE(Array) { K_UNREACHABLE(); };
 
-                float f = GetNumber<float>(value);
+        CASE(Float32) {
+            const ParameterInfo &param = func->parameters[i];
+            Napi::Value value = info[param.offset];
 
-                if (param.vec_count) [[likely]] {
-                    memset((uint8_t *)vec_ptr + 4, 0, 4);
-                    *(float *)(vec_ptr++) = f;
+            float f;
+            if (!TryNumber(value, &f)) [[unlikely]] {
+                ThrowError<Napi::TypeError>(env, "Unexpected %1 value, expected number", GetValueType(instance, value));
+                return false;
+            }
+
+            if (param.vec_count) [[likely]] {
+                memset((uint8_t *)vec_ptr + 4, 0, 4);
+                *(float *)(vec_ptr++) = f;
 #if defined(_WIN32)
-                } else if (param.gpr_count) {
-                    memset((uint8_t *)gpr_ptr + 4, 0, 4);
-                    *(float *)(gpr_ptr++) = f;
+            } else if (param.gpr_count) {
+                memset((uint8_t *)gpr_ptr + 4, 0, 4);
+                *(float *)(gpr_ptr++) = f;
 #endif
-                } else {
+            } else {
 #if defined(__APPLE__)
-                    args_ptr = AlignUp(args_ptr, 4);
-                    *(float *)args_ptr = f;
-                    args_ptr = (uint64_t *)((uint8_t *)args_ptr + 4);
+                args_ptr = AlignUp(args_ptr, 4);
+                *(float *)args_ptr = f;
+                args_ptr = (uint64_t *)((uint8_t *)args_ptr + 4);
 #else
-                    memset((uint8_t *)args_ptr + 4, 0, 4);
-                    *(float *)(args_ptr++) = f;
+                memset((uint8_t *)args_ptr + 4, 0, 4);
+                *(float *)(args_ptr++) = f;
 #endif
-                }
-            } break;
-            case PrimitiveKind::Float64: {
-                if (!value.IsNumber() && !value.IsBigInt()) [[unlikely]] {
-                    ThrowError<Napi::TypeError>(env, "Unexpected %1 value, expected number", GetValueType(instance, value));
-                    return false;
-                }
+            }
+        };
+        CASE(Float64) {
+            const ParameterInfo &param = func->parameters[i];
+            Napi::Value value = info[param.offset];
 
-                double d = GetNumber<double>(value);
+            double d;
+            if (!TryNumber(value, &d)) [[unlikely]] {
+                ThrowError<Napi::TypeError>(env, "Unexpected %1 value, expected number", GetValueType(instance, value));
+                return false;
+            }
 
-                if (param.vec_count) [[likely]] {
-                    *(double *)(vec_ptr++) = d;
+            if (param.vec_count) [[likely]] {
+                *(double *)(vec_ptr++) = d;
 #if defined(_WIN32)
-                } else if (param.gpr_count) {
-                    *(double *)(gpr_ptr++) = d;
+            } else if (param.gpr_count) {
+                *(double *)(gpr_ptr++) = d;
 #endif
-                } else {
+            } else {
 #if defined(__APPLE__)
-                    args_ptr = AlignUp(args_ptr, 8);
+                args_ptr = AlignUp(args_ptr, 8);
 #endif
-                    *(double *)(args_ptr++) = d;
-                }
-            } break;
-            case PrimitiveKind::Callback: {
-                void *ptr;
-                if (!PushCallback(value, param.type, &ptr)) [[unlikely]]
-                    return false;
+                *(double *)(args_ptr++) = d;
+            }
+        };
+
+        CASE(Callback) {
+            const ParameterInfo &param = func->parameters[i];
+            Napi::Value value = info[param.offset];
+
+            void *ptr;
+            if (!PushCallback(value, param.type, &ptr)) [[unlikely]]
+                return false;
 
 #if defined(__APPLE__)
-                args_ptr = param.gpr_count ? args_ptr : AlignUp(args_ptr, 8);
+            args_ptr = param.gpr_count ? args_ptr : AlignUp(args_ptr, 8);
 #endif
-                *(void **)((param.gpr_count ? gpr_ptr : args_ptr)++) = ptr;
-            } break;
+            *(void **)((param.gpr_count ? gpr_ptr : args_ptr)++) = ptr;
+        };
 
-            case PrimitiveKind::Prototype: { K_UNREACHABLE(); } break;
-        }
+        CASE(Prototype) { /* End loop */ };
     }
 
 #undef PUSH_INTEGER_SWAP
 #undef PUSH_INTEGER
 
+#undef OR
+#undef CASE
+#undef LOOP
+
     new_sp = mem->stack.end();
 
     return true;
@@ -653,17 +710,8 @@ Napi::Value CallData::Complete(const FunctionInfo *func)
         case PrimitiveKind::String: return result.ptr ? Napi::String::New(env, (const char *)result.ptr) : env.Null();
         case PrimitiveKind::String16: return result.ptr ? Napi::String::New(env, (const char16_t *)result.ptr) : env.Null();
         case PrimitiveKind::String32: return result.ptr ? MakeStringFromUTF32(env, (const char32_t *)result.ptr) : env.Null();
-        case PrimitiveKind::Pointer:
-        case PrimitiveKind::Callback: {
-            if (result.ptr) {
-                Napi::External<void> external = Napi::External<void>::New(env, result.ptr);
-                SetValueTag(external, func->ret.type->ref.marker);
-
-                return external;
-            } else {
-                return env.Null();
-            }
-        } break;
+        case PrimitiveKind::Pointer: return result.ptr ? WrapPointer(env, func->ret.type->ref.type, result.ptr) : env.Null();
+        case PrimitiveKind::Callback: return result.ptr ? WrapCallback(env, func->ret.type->ref.type, result.ptr) : env.Null();
         case PrimitiveKind::Record:
         case PrimitiveKind::Union: {
             if (func->ret.vec_count) { // HFA
@@ -1026,22 +1074,29 @@ void CallData::Relay(Size idx, uint8_t *own_sp, uint8_t *caller_sp, bool switch_
                 Napi::Value arg = str32 ? MakeStringFromUTF32(env, str32) : env.Null();
                 arguments.Append(arg);
             } break;
-            case PrimitiveKind::Pointer:
-            case PrimitiveKind::Callback: {
+            case PrimitiveKind::Pointer: {
 #if defined(__APPLE__)
                 args_ptr = AlignUp(args_ptr, 8);
 #endif
 
                 void *ptr2 = *(void **)((param.gpr_count ? gpr_ptr : args_ptr)++);
 
-                if (ptr2) {
-                    Napi::External<void> external = Napi::External<void>::New(env, ptr2);
-                    SetValueTag(external, param.type->ref.marker);
+                Napi::Value p = ptr2 ? WrapPointer(env, param.type->ref.type, ptr2) : env.Null();
+                arguments.Append(p);
 
-                    arguments.Append(external);
-                } else {
-                    arguments.Append(env.Null());
+                if (param.type->dispose) {
+                    param.type->dispose(env, param.type, ptr2);
                 }
+            } break;
+            case PrimitiveKind::Callback: {
+#if defined(__APPLE__)
+                args_ptr = AlignUp(args_ptr, 8);
+#endif
+
+                void *ptr2 = *(void **)((param.gpr_count ? gpr_ptr : args_ptr)++);
+
+                Napi::Value p = ptr2 ? WrapCallback(env, param.type->ref.type, ptr2) : env.Null();
+                arguments.Append(p);
 
                 if (param.type->dispose) {
                     param.type->dispose(env, param.type, ptr2);
@@ -1148,22 +1203,22 @@ void CallData::Relay(Size idx, uint8_t *own_sp, uint8_t *caller_sp, bool switch_
 
 #define RETURN_INTEGER(CType) \
         do { \
-            if (!value.IsNumber() && !value.IsBigInt()) [[unlikely]] { \
+            CType v; \
+            if (!TryNumber(value, &v)) [[unlikely]] { \
                 ThrowError<Napi::TypeError>(env, "Unexpected %1 value, expected number", GetValueType(instance, value)); \
                 return; \
             } \
              \
-            CType v = GetNumber<CType>(value); \
             out_reg->x0 = (uint64_t)v; \
         } while (false)
 #define RETURN_INTEGER_SWAP(CType) \
         do { \
-            if (!value.IsNumber() && !value.IsBigInt()) [[unlikely]] { \
+            CType v; \
+            if (!TryNumber(value, &v)) [[unlikely]] { \
                 ThrowError<Napi::TypeError>(env, "Unexpected %1 value, expected number", GetValueType(instance, value)); \
                 return; \
             } \
              \
-            CType v = GetNumber<CType>(value); \
             out_reg->x0 = (uint64_t)ReverseBytes(v); \
         } while (false)
 
@@ -1171,12 +1226,12 @@ void CallData::Relay(Size idx, uint8_t *own_sp, uint8_t *caller_sp, bool switch_
     switch (type->primitive) {
         case PrimitiveKind::Void: {} break;
         case PrimitiveKind::Bool: {
-            if (!value.IsBoolean()) [[unlikely]] {
+            bool b;
+            if (napi_get_value_bool(env, value, &b) != napi_ok) [[unlikely]] {
                 ThrowError<Napi::TypeError>(env, "Unexpected %1 value, expected boolean", GetValueType(instance, value));
                 return;
             }
 
-            bool b = value.As<Napi::Boolean>();
             out_reg->x0 = (uint64_t)b;
         } break;
         case PrimitiveKind::Int8: { RETURN_INTEGER(int8_t); } break;
@@ -1215,24 +1270,9 @@ void CallData::Relay(Size idx, uint8_t *own_sp, uint8_t *caller_sp, bool switch_
             out_reg->x0 = (uint64_t)str32;
         } break;
         case PrimitiveKind::Pointer: {
-            uint8_t *ptr;
-
-            if (CheckValueTag(value, type->ref.marker)) {
-                ptr = value.As<Napi::External<uint8_t>>().Data();
-            } else if (IsObject(value) && (type->ref.type->primitive == PrimitiveKind::Record ||
-                                           type->ref.type->primitive == PrimitiveKind::Union)) {
-                Napi::Object obj = value.As<Napi::Object>();
-
-                ptr = AllocHeap(type->ref.type->size, 16);
-
-                if (!PushObject(obj, type->ref.type, ptr))
-                    return;
-            } else if (IsNullOrUndefined(value)) {
-                ptr = nullptr;
-            } else {
-                ThrowError<Napi::TypeError>(env, "Unexpected %1 value, expected %2", GetValueType(instance, value), type->name);
+            void *ptr;
+            if (!PushPointer(value, type, 1, &ptr)) [[unlikely]]
                 return;
-            }
 
             out_reg->x0 = (uint64_t)ptr;
         } break;
@@ -1260,42 +1300,30 @@ void CallData::Relay(Size idx, uint8_t *own_sp, uint8_t *caller_sp, bool switch_
         } break;
         case PrimitiveKind::Array: { K_UNREACHABLE(); } break;
         case PrimitiveKind::Float32: {
-            if (!value.IsNumber() && !value.IsBigInt()) [[unlikely]] {
+            float f;
+
+            if (!TryNumber(value, &f)) [[unlikely]] {
                 ThrowError<Napi::TypeError>(env, "Unexpected %1 value, expected number", GetValueType(instance, value));
                 return;
             }
 
-            float f = GetNumber<float>(value);
-
             memset((uint8_t *)&out_reg->d0 + 4, 0, 4);
             memcpy(&out_reg->d0, &f, 4);
         } break;
         case PrimitiveKind::Float64: {
-            if (!value.IsNumber() && !value.IsBigInt()) [[unlikely]] {
+            double d;
+
+            if (!TryNumber(value, &d)) [[unlikely]] {
                 ThrowError<Napi::TypeError>(env, "Unexpected %1 value, expected number", GetValueType(instance, value));
                 return;
             }
 
-            double d = GetNumber<double>(value);
             out_reg->d0 = d;
         } break;
         case PrimitiveKind::Callback: {
             void *ptr;
-
-            if (value.IsFunction()) {
-                Napi::Function func2 = value.As<Napi::Function>();
-
-                ptr = ReserveTrampoline(type->ref.proto, func2);
-                if (!ptr) [[unlikely]]
-                    return;
-            } else if (CheckValueTag(value, type->ref.marker)) {
-                ptr = value.As<Napi::External<void>>().Data();
-            } else if (IsNullOrUndefined(value)) {
-                ptr = nullptr;
-            } else {
-                ThrowError<Napi::TypeError>(env, "Unexpected %1 value, expected %2", GetValueType(instance, value), type->name);
+            if (!PushCallback(value, type, &ptr)) [[unlikely]]
                 return;
-            }
 
             out_reg->x0 = (uint64_t)ptr;
         } break;
@@ -1309,12 +1337,6 @@ void CallData::Relay(Size idx, uint8_t *own_sp, uint8_t *caller_sp, bool switch_
     err_guard.Disable();
 }
 
-void *GetTrampoline(int16_t idx, const FunctionInfo *proto)
-{
-    bool vec = proto->forward_fp || IsFloat(proto->ret.type);
-    return Trampolines[idx][vec];
-}
-
 }
 
 #endif