koffi 1.1.2 → 1.1.5

package/CMakeLists.txt

@@ -18,11 +18,10 @@ find_package(CNoke)
 
 set(CMAKE_CXX_STANDARD 17)
 if(MSVC)
-    add_compile_options(/W4 /wd4200 /wd4458 /wd4706 /wd4100 /wd4127 /wd4702)
+    add_compile_options(/W4 /wd4200 /wd4458 /wd4706 /wd4100 /wd4127 /wd4702 /wd4201)
     enable_language(ASM_MASM)
 else()
-    add_compile_options(-Wall -Wextra -Wno-missing-field-initializers
-                        -Wno-unused-parameter -Wno-class-memaccess)
+    add_compile_options(-Wall -Wextra -Wno-missing-field-initializers -Wno-unused-parameter)
     if(CMAKE_CXX_COMPILER_ID MATCHES "Clang")
         add_compile_options(-Wno-unknown-warning-option)
     endif()
@@ -32,31 +31,36 @@ endif()
 
 set(KOFFI_SRC
     src/call.cc
-    src/abi_arm32.cc
-    src/abi_arm64.cc
-    src/abi_x64_sysv.cc
-    src/abi_x64_win.cc
-    src/abi_x86.cc
     src/ffi.cc
     src/parser.cc
     src/util.cc
     vendor/libcc/libcc.cc
 )
 if(CMAKE_SIZEOF_VOID_P EQUAL 8)
-    if(WIN32)
-        list(APPEND KOFFI_SRC src/abi_x64_win_fwd.asm)
-    elseif(CMAKE_SYSTEM_PROCESSOR MATCHES "aarch64|arm64")
-        list(APPEND KOFFI_SRC src/abi_arm64_fwd.S)
+    if(CMAKE_SYSTEM_PROCESSOR MATCHES "aarch|arm")
+        if(WIN32)
+            list(APPEND KOFFI_SRC src/abi_arm64.cc src/abi_arm64_fwd.asm)
+        else()
+            list(APPEND KOFFI_SRC src/abi_arm64.cc src/abi_arm64_fwd.S)
+        endif()
+    elseif(CMAKE_SYSTEM_PROCESSOR MATCHES "riscv")
+        list(APPEND KOFFI_SRC src/abi_riscv64.cc src/abi_riscv64_fwd.S)
     else()
-        list(APPEND KOFFI_SRC src/abi_x64_sysv_fwd.S)
+        if(WIN32)
+            list(APPEND KOFFI_SRC src/abi_x64_win.cc src/abi_x64_win_fwd.asm)
+        else()
+            list(APPEND KOFFI_SRC src/abi_x64_sysv.cc src/abi_x64_sysv_fwd.S)
+        endif()
     endif()
 elseif(CMAKE_SIZEOF_VOID_P EQUAL 4)
-    if(WIN32)
-        list(APPEND KOFFI_SRC src/abi_x86_fwd.asm)
-    elseif(CMAKE_SYSTEM_PROCESSOR MATCHES "i[3456]86|x86|AMD64")
-        list(APPEND KOFFI_SRC src/abi_x86_fwd.S)
-    elseif(CMAKE_SYSTEM_PROCESSOR MATCHES "armv[678]l")
-        list(APPEND KOFFI_SRC src/abi_arm32_fwd.S)
+    if(CMAKE_SYSTEM_PROCESSOR MATCHES "arm")
+        list(APPEND KOFFI_SRC src/abi_arm32.cc src/abi_arm32_fwd.S)
+    else()
+        if(WIN32)
+            list(APPEND KOFFI_SRC src/abi_x86.cc src/abi_x86_fwd.asm)
+        else()
+            list(APPEND KOFFI_SRC src/abi_x86.cc src/abi_x86_fwd.S)
+        endif()
     endif()
 endif()
 

package/README.md

@@ -13,43 +13,32 @@
   * [Raylib results](#raylib-results)
 - [Tests](#tests)
 - [Compilation](#compilation)
-  * [Windows](#windows)
+  * [Windows](#windows-1)
   * [Other platforms](#other-platforms)
 
 # Introduction
 
-Koffi is a fast and easy-to-use FFI module for Node.js, with support for primitive and aggregate data types (structs), both by reference (pointer) and by value.
+Koffi is a fast and easy-to-use C FFI module for Node.js, with support for primitive and aggregate data types (structs and fixed-size arrays), both by reference (pointer) and by value.
 
 The following features are planned in the near future:
 
 * 1.2: C to JS callbacks
 * 1.3: Type parser
 
-The following platforms __are officially supported and tested__ at the moment:
-
-Platform  | Architecture                     | Sync calls | Async calls | Callbacks | Pre-built binary
---------- | -------------------------------- | ---------- | ----------- | --------- | ----------------
-Windows   | x86 (cdecl, stdcall, fastcall)   | 🟩 Yes      | 🟩 Yes       | 🟥 No      | 🟩 Yes
-Windows   | x86_64                           | 🟩 Yes      | 🟩 Yes       | 🟥 No      | 🟩 Yes
-Linux     | x86                              | 🟩 Yes      | 🟩 Yes       | 🟥 No      | 🟩 Yes
-Linux     | x86_64                           | 🟩 Yes      | 🟩 Yes       | 🟥 No      | 🟩 Yes
-Linux     | ARM32+VFP Little Endian          | 🟩 Yes      | 🟩 Yes       | 🟥 No      | 🟩 Yes
-Linux     | ARM64 Little Endian              | 🟩 Yes      | 🟩 Yes       | 🟥 No      | 🟩 Yes
-FreeBSD   | x86                              | 🟩 Yes      | 🟩 Yes       | 🟥 No      | 🟩 Yes
-FreeBSD   | x86_64                           | 🟩 Yes      | 🟩 Yes       | 🟥 No      | 🟩 Yes
-FreeBSD   | ARM64 Little Endian              | 🟩 Yes      | 🟩 Yes       | 🟥 No      | 🟩 Yes
-macOS     | x86_64                           | 🟩 Yes      | 🟩 Yes       | 🟥 No      | 🟩 Yes
-macOS     | ARM64 (M1) Little Endian         | 🟩 Yes      | 🟩 Yes       | 🟥 No      | 🟥 No
-OpenBSD   | x86_64                           | 🟧 Maybe    | 🟧 Maybe     | 🟥 No      | 🟥 No
-OpenBSD   | x86                              | 🟧 Maybe    | 🟧 Maybe     | 🟥 No      | 🟥 No
-OpenBSD   | ARM64 Little Endian              | 🟧 Maybe    | 🟧 Maybe     | 🟥 No      | 🟥 No
-NetBSD    | x86_64                           | 🟧 Maybe    | 🟧 Maybe     | 🟥 No      | 🟥 No
-NetBSD    | x86                              | 🟧 Maybe    | 🟧 Maybe     | 🟥 No      | 🟥 No
-NetBSD    | ARM64 Little Endian              | 🟧 Maybe    | 🟧 Maybe     | 🟥 No      | 🟥 No
-
-🟩 Tested, fully operational
-🟧 May work, but not actively tested
-🟥 Does not work yet
+The following combinations of OS and architectures __are officially supported and tested__ at the moment:
+
+ISA / OS           | Windows     | Linux    | macOS       | FreeBSD     | OpenBSD
+------------------ | ----------- | -------- | ----------- | ----------- | --------
+x86 (IA32) [^1]    | 🟩 Yes      | 🟩 Yes   | ⬜️ *N/A*    | 🟩 Yes      | 🟩 Yes
+x86_64 (AMD64)     | 🟩 Yes      | 🟩 Yes   | 🟩 Yes      | 🟩 Yes      | 🟩 Yes
+ARM32 LE [^2]      | ⬜️ *N/A*    | 🟩 Yes   | ⬜️ *N/A*    | 🟨 Probably | 🟨 Probably
+ARM64 (AArch64) LE | 🟧 Maybe    | 🟩 Yes   | 🟩 Yes [^3] | 🟩 Yes      | 🟨 Probably
+RISC-V 64 [^4]     | ⬜️ *N/A*    | 🟩 Yes   | ⬜️ *N/A*    | 🟨 Probably | 🟨 Probably
+
+[^1]: The following call conventions are supported: cdecl, stdcall, MS fastcall, thiscall.
+[^2]: The prebuilt binary uses the hard float ABI and expects a VFP coprocessor. Build from source to use Koffi with a different ABI (softfp, soft).
+[^3]: However, we don't provide prebuilt binaries for macOS on Apple M1.
+[^4]: Only the LP64D (double-precision float) ABI gets tested. The LP64 ABI is supported in theory (untested), the LP64F ABI is not supported.
 
 This is still in development, bugs are to be expected. More tests will come in the near future.
 

package/benchmark/CMakeLists.txt

@@ -12,12 +12,10 @@
 # along with this program. If not, see https://www.gnu.org/licenses/.
 
 cmake_minimum_required(VERSION 3.12)
-project(koffi C CXX ASM)
+project(koffi_benchmark C CXX)
 
 find_package(CNoke)
 
-set(CMAKE_CXX_STANDARD 20)
-
 set(THREADS_PREFER_PTHREAD_FLAG ON)
 find_package(Threads REQUIRED)
 
@@ -26,6 +24,16 @@ if(NOT TARGET koffi)
 endif()
 add_subdirectory(../test test)
 
+set(CMAKE_CXX_STANDARD 17)
+if(MSVC)
+    add_compile_options(/W4 /wd4200 /wd4458 /wd4706 /wd4100 /wd4127 /wd4702 /wd4201)
+else()
+    add_compile_options(-Wall -Wextra -Wno-missing-field-initializers -Wno-unused-parameter)
+    if(CMAKE_CXX_COMPILER_ID MATCHES "Clang")
+        add_compile_options(-Wno-unknown-warning-option)
+    endif()
+endif()
+
 # ---- atoi ----
 
 add_executable(atoi_cc atoi_cc.cc ../vendor/libcc/libcc.cc)

package/benchmark/raylib_cc.cc

@@ -31,28 +31,28 @@ int Main(int argc, char **argv)
     SetWindowState(FLAG_WINDOW_HIDDEN);
     InitWindow(640, 480, "Raylib Test");
 
-    Image img = GenImageColor(800, 600, Color { .r = 0, .g = 0, .b = 0, .a = 255 });
+    Image img = GenImageColor(800, 600, Color { 0, 0, 0, 255 });
     Font font = GetFontDefault();
 
     int64_t start = GetMonotonicTime();
 
     for (int i = 0; i < iterations; i++) {
-        ImageClearBackground(&img, Color { .r = 0, .g = 0, .b = 0, .a = 255 });
+        ImageClearBackground(&img, Color { 0, 0, 0, 255 });
 
         for (int j = 0; j < 3600; j++) {
             const char *text = "Hello World!";
-            int text_width = MeasureTextEx(font, text, 10, 1).x;
+            float text_width = MeasureTextEx(font, text, 10, 1).x;
 
             double angle = (j * 7) * PI / 180;
             Color color = {
-                .r = (unsigned char)(127.5 + 127.5 * sin(angle)),
-                .g = (unsigned char)(127.5 + 127.5 * sin(angle + PI / 2)),
-                .b = (unsigned char)(127.5 + 127.5 * sin(angle + PI)),
-                .a = 255
+                (unsigned char)(127.5 + 127.5 * sin(angle)),
+                (unsigned char)(127.5 + 127.5 * sin(angle + PI / 2)),
+                (unsigned char)(127.5 + 127.5 * sin(angle + PI)),
+                255
             };
             Vector2 pos = {
-                .x = (float)((img.width / 2 - text_width / 2) + j * 0.1 * cos(angle - PI / 2)),
-                .y = (float)((img.height / 2 - 16) + j * 0.1 * sin(angle - PI / 2))
+                (float)((img.width / 2 - text_width / 2) + j * 0.1 * cos(angle - PI / 2)),
+                (float)((img.height / 2 - 16) + j * 0.1 * sin(angle - PI / 2))
             };
 
             ImageDrawTextEx(&img, font, text, pos, 10, 1, color);

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "koffi",
-  "version": "1.1.2",
-  "description": "Fast and simple FFI (foreign function interface) for Node.js",
+  "version": "1.1.5",
+  "description": "Fast and simple C FFI (foreign function interface) for Node.js",
   "keywords": [
     "foreign",
     "function",

package/qemu/qemu.js

@@ -124,13 +124,12 @@ async function main() {
             machine.started = false;
 
             machine.qemu.accelerate = null;
-            if (accelerate) {
+            if (accelerate && (machine.qemu.binary == 'qemu-system-x86_64' ||
+                               machine.qemu.binary == 'qemu-system-i386')) {
                 if (process.platform == 'linux') {
-                    if (process.arch == 'x64' && (machine.info.arch == 'x64' || machine.info.arch == 'ia32'))
-                        machine.qemu.accelerate = 'kvm';
+                    machine.qemu.accelerate = 'kvm';
                 } else if (process.platform == 'win32') {
-                    if (process.arch == 'x64' && machine.info.arch == 'x64')
-                        machine.qemu.accelerate = 'whpx';
+                    machine.qemu.accelerate = 'whpx';
                 }
             }
         }
@@ -141,19 +140,23 @@ async function main() {
 
         for (let pattern of patterns) {
             let re = minimatch.makeRe(pattern);
+            let match = false;
 
             for (let name in machines_map) {
                 let machine = machines_map[name];
 
-                if (name.match(re) || machine.name.match(re))
+                if (name.match(re) || machine.name.match(re)) {
                     machines.add(name);
+                    match = true;
+                }
             }
-        }
 
-        if (!machines.size) {
-            console.log('Could not match any machine');
-            process.exit(1);
+            if (!match) {
+                console.log(`Pattern '${pattern}' does not match any machine`);
+                process.exit(1);
+            }
         }
+
     } else {
         machines = new Set(Object.keys(machines_map));
 
@@ -300,6 +303,8 @@ async function pack() {
             ignore.add(machine);
     }
 
+    let ready = ignore.size;
+
     success &= await start(false);
     success &= await copy(machine => Object.values(machine.builds).map(build => build.directory));
 
@@ -402,6 +407,9 @@ async function pack() {
     if (machines.some(machine => machine.started))
         success &= await stop(false);
 
+    if (ignore.size > ready)
+        throw new Error('Some machines are missing, refusing to pack');
+
     return success;
 }
 

package/qemu/registry/machines.json

@@ -253,7 +253,7 @@
         },
 
         "info": {
-            "version": 1,
+            "version": 2,
             "platform": "freebsd",
             "arch": "x64",
 
@@ -298,7 +298,7 @@
         },
 
         "info": {
-            "version": 1,
+            "version": 2,
             "platform": "freebsd",
             "arch": "ia32",
 
@@ -343,7 +343,7 @@
         },
 
         "info": {
-            "version": 1,
+            "version": 2,
             "platform": "freebsd",
             "arch": "arm64",
 
@@ -421,5 +421,140 @@
                 }
             }
         }
+    },
+
+    "openbsd_x64": {
+        "name": "OpenBSD x64",
+
+        "qemu": {
+            "binary": "qemu-system-x86_64",
+            "arguments": ["-m", "1G", "-smp", 2, "-hda", "disk.qcow2", "-netdev", "user,id=mynet,hostfwd=tcp::22210-:22", "-device", "e1000,netdev=mynet", "-vnc", "127.0.0.1:20"]
+        },
+
+        "info": {
+            "version": 1,
+            "platform": "openbsd",
+            "arch": "x64",
+
+            "ssh_port": 22210,
+            "vnc_port": 5920,
+
+            "username": "openbsd",
+            "password": "openbsd",
+
+            "shutdown": "sudo shutdown -p now"
+        },
+
+        "builds": {
+            "OpenBSD x64": {
+                "directory": "/home/openbsd/luigi",
+                "build": "node ../cnoke/cnoke.js"
+            }
+        },
+
+        "tests": {
+            "OpenBSD x64": {
+                "directory": "/home/openbsd/luigi",
+                "build": {
+                    "Build": "node ../cnoke/cnoke.js -C test"
+                },
+                "commands": {
+                    "Test Sync": "node test/sync.js",
+                    "Test Async": "node test/async.js",
+                    "Test Raylib": "xvfb-run node test/raylib.js",
+                    "Test SQLite": "node test/sqlite.js"
+                }
+            }
+        }
+    },
+
+    "openbsd_i386": {
+        "name": "OpenBSD i386",
+
+        "qemu": {
+            "binary": "qemu-system-x86_64",
+            "arguments": ["-m", "1G", "-smp", 2, "-hda", "disk.qcow2", "-netdev", "user,id=mynet,hostfwd=tcp::22211-:22", "-device", "e1000,netdev=mynet", "-vnc", "127.0.0.1:21"]
+        },
+
+        "info": {
+            "version": 1,
+            "platform": "openbsd",
+            "arch": "ia32",
+
+            "ssh_port": 22211,
+            "vnc_port": 5921,
+
+            "username": "openbsd",
+            "password": "openbsd",
+
+            "shutdown": "sudo shutdown -p now"
+        },
+
+        "builds": {
+            "OpenBSD i386": {
+                "directory": "/home/openbsd/luigi",
+                "build": "node ../cnoke/cnoke.js"
+            }
+        },
+
+        "tests": {
+            "OpenBSD i386": {
+                "directory": "/home/openbsd/luigi",
+                "build": {
+                    "Build": "node ../cnoke/cnoke.js -C test"
+                },
+                "commands": {
+                    "Test Sync": "node test/sync.js",
+                    "Test Async": "node test/async.js",
+                    "Test Raylib": "xvfb-run node test/raylib.js",
+                    "Test SQLite": "node test/sqlite.js"
+                }
+            }
+        }
+    },
+
+    "debian_riscv64": {
+        "name": "Debian RISC-V 64",
+
+        "qemu": {
+            "binary": "qemu-system-riscv64",
+            "arguments": ["-machine", "virt", "-cpu", "rv64", "-m", "1G", "-smp", 2, "-device", "virtio-blk-device,drive=hd", "-drive", "file=disk.qcow2,if=none,id=hd", "-device", "virtio-net-device,netdev=net", "-netdev", "user,id=net,hostfwd=tcp::22212-:22", "-bios", "fw_jump.elf", "-kernel", "uboot.elf", "-append", "root=LABEL=rootfs console=ttyS0", "-vnc", "127.0.0.1:22"]
+        },
+
+        "info": {
+            "version": 1,
+            "platform": "linux",
+            "arch": "riscv64",
+
+            "ssh_port": 22212,
+            "vnc_port": 5912,
+
+            "username": "debian",
+            "password": "debian",
+
+            "shutdown": "sudo poweroff"
+        },
+
+        "builds": {
+            "Linux RISC-V 64": {
+                "directory": "/home/debian/luigi",
+                "build": "node ../cnoke/cnoke.js"
+            }
+        },
+
+        "tests": {
+            "Linux RISC-V 64": {
+                "directory": "/home/debian/luigi",
+                "build": {
+                    "Build": "node ../cnoke/cnoke.js -C test"
+                },
+                "commands": {
+                    "Test Sync": "node test/sync.js",
+                    "Test Async": "node test/async.js",
+                    "Test Raylib": "xvfb-run node test/raylib.js",
+                    "Test SQLite": "node test/sqlite.js"
+                }
+            }
+        }
     }
 }

package/qemu/registry/sha256sum.txt

@@ -22,20 +22,20 @@ ed05b84b7759ec945bb5422305a3a3c5b61d89e231844656ea31e07d89451a20 *qemu/debian_i3
 292a61c415a99a9c63a9337f1e074bea6c5df594e16bc40ded76141963c67ea4 *qemu/debian_i386/install.sh
 8dc1360e1c23ea21931f5eb94461d15fac6bcec00cf42bf1e590b7fb937e80c1 *qemu/debian_i386/initrd.img-5.10.0-12-686-pae
 338602d969f953cd88c2df736b9f8f7ee53029d233c83c7398258ae5bcb4e286 *qemu/debian_i386/disk.qcow2
-6b86b273ff34fce19d6b804eff5a3f5747ada4eaa22f1d49c01e52ddb7875b4b *qemu/freebsd_arm64/VERSION
-5daea533330eb8d2f8df946561680e09f8dacc467fbf03d28caa2115d94386b5 *qemu/freebsd_arm64/QEMU_EFI.img
-a083e4e1f58f477e4a9e8ae30f75660e82275b88f77b5e2c7f5d6ee1deb43b41 *qemu/freebsd_arm64/varstore.img
+e5d9d3e1e4a5c980fce4ef3aab5e29b9607b8e5da6e9fe58c3f396009e283cff *qemu/freebsd_arm64/disk.qcow2
 53a4dc48317e1c9de4a088cd45ba39a1810f0bafad75c7f810e18d4e3eb340e3 *qemu/freebsd_arm64/install.sh
-a19ad06aec7972f3b95a56fb9bb8ac8a6afcd61b586f398691d58c704e8c466e *qemu/freebsd_arm64/disk.qcow2
-10f2bc603b171368cacda43485f8f72567af0a9a490b7e0fa0c14c983ae66f16 *qemu/freebsd_arm64/xvfb-run.sh
-6b86b273ff34fce19d6b804eff5a3f5747ada4eaa22f1d49c01e52ddb7875b4b *qemu/freebsd_x64/VERSION
-06eef544fe9c61d3905b0e588c306d1dedac7ee7d4c01df154ff6ed64e25ef77 *qemu/freebsd_x64/install.sh
-060b66901313ddfcd434c20a7b13052b3e5126e43c4e44884b85191560bd99eb *qemu/freebsd_x64/disk.qcow2
-10f2bc603b171368cacda43485f8f72567af0a9a490b7e0fa0c14c983ae66f16 *qemu/freebsd_x64/xvfb-run.sh
-6b86b273ff34fce19d6b804eff5a3f5747ada4eaa22f1d49c01e52ddb7875b4b *qemu/freebsd_i386/VERSION
+5daea533330eb8d2f8df946561680e09f8dacc467fbf03d28caa2115d94386b5 *qemu/freebsd_arm64/QEMU_EFI.img
+8ac20d46117a5a15761bf3070e365141b96ee3283250824f113029e106c62a50 *qemu/freebsd_arm64/varstore.img
+d4735e3a265e16eee03f59718b9b5d03019c07d8b6c51f90da3a666eec13ab35 *qemu/freebsd_arm64/VERSION
+73837b3991eae13f6c346955ea4aa419c8c168cadf70e0a38446db60cf47b0e9 *qemu/freebsd_arm64/xvfb-run.sh
+fd0e51c43d7848564a81ddcb3eecf49103251bffcf25315ef87db01350ad4ac5 *qemu/freebsd_i386/disk.qcow2
 bdeac2cca1d22d70c2d388a50709d5cba3069bbd4bd1bcb102955b0ee12ca3e9 *qemu/freebsd_i386/install.sh
-8191cea63d738cdca6df0b2d2d6953675e1dd33cac4d98fff88ba4367b182b64 *qemu/freebsd_i386/disk.qcow2
-10f2bc603b171368cacda43485f8f72567af0a9a490b7e0fa0c14c983ae66f16 *qemu/freebsd_i386/xvfb-run.sh
+d4735e3a265e16eee03f59718b9b5d03019c07d8b6c51f90da3a666eec13ab35 *qemu/freebsd_i386/VERSION
+73837b3991eae13f6c346955ea4aa419c8c168cadf70e0a38446db60cf47b0e9 *qemu/freebsd_i386/xvfb-run.sh
+41649d2f5959388e1c5d8ee89bb084795470a8b9e68d87861b4e95e4102b62cf *qemu/freebsd_x64/disk.qcow2
+06eef544fe9c61d3905b0e588c306d1dedac7ee7d4c01df154ff6ed64e25ef77 *qemu/freebsd_x64/install.sh
+d4735e3a265e16eee03f59718b9b5d03019c07d8b6c51f90da3a666eec13ab35 *qemu/freebsd_x64/VERSION
+73837b3991eae13f6c346955ea4aa419c8c168cadf70e0a38446db60cf47b0e9 *qemu/freebsd_x64/xvfb-run.sh
 16a4f09b3e70b97f5cfb1cf9b913d67d0ec45c4342d202cc9a2b2cfee852a8de *qemu/macos_x64/ESP.qcow2
 5d2ac383371b408398accee7ec27c8c09ea5b74a0de0ceea6513388b15be5d1e *qemu/macos_x64/OVMF_VARS.fd
 d79538ac489f1948b04b65f12c5618c28e4a5de0be062f5cf1d73e422a091a37 *qemu/macos_x64/install.txt
@@ -43,3 +43,18 @@ d79538ac489f1948b04b65f12c5618c28e4a5de0be062f5cf1d73e422a091a37 *qemu/macos_x64
 5cc2d42949c7e4e763db0abeb88299972bafef991d87dbf744a5108ef9190f6f *qemu/macos_x64/OVMF_VARS-1024x768.fd
 53c234e5e8472b6ac51c1ae1cab3fe06fad053beb8ebfd8977b010655bfdd3c3 *qemu/macos_x64/VERSION
 2a247500e8ad9aa479994097ba5976bd881fde4de1516abd14491f3dba9fe060 *qemu/macos_x64/OVMF_CODE.fd
+3a3395f70240d8e3137dfe06ef7dd76255dc597626e087d33abdcc65710f3dff *qemu/openbsd_x64/disk.qcow2
+9dcdabfc12ad2ba2da1b9c3d95dc46311a12d6e3190f1bfd0c9b184ac1577c94 *qemu/openbsd_x64/install.sh
+6b86b273ff34fce19d6b804eff5a3f5747ada4eaa22f1d49c01e52ddb7875b4b *qemu/openbsd_x64/VERSION
+7b66bdf8b77fd1c6256ffdab623e37ce099797ad657dcd25af5df0486fadce37 *qemu/openbsd_x64/xvfb-run.sh
+570bc6da16ca3b2b068963e7960e76fd23df3ada54398dfc6b6ddf97348ef5d7 *qemu/openbsd_i386/disk.qcow2
+b82c940781ee678499851e1d2aa4868b7154a16ecc4f01c6c56940276c02748f *qemu/openbsd_i386/install.sh
+6b86b273ff34fce19d6b804eff5a3f5747ada4eaa22f1d49c01e52ddb7875b4b *qemu/openbsd_i386/VERSION
+7b66bdf8b77fd1c6256ffdab623e37ce099797ad657dcd25af5df0486fadce37 *qemu/openbsd_i386/xvfb-run.sh
+64048432020bda960f397b5a138ea98aa6bbd42db02e8bb34ff3b9320763e36d *qemu/debian_riscv64/uboot.elf
+0bdd93879f87c338d62422a65fdcba4d8c834848f2305f31b916539cb523e9e8 *qemu/debian_riscv64/readme.txt
+76a5819cdacc8fe284370f441820620e911877122068a3b391fa064b7e9eb180 *qemu/debian_riscv64/initrd
+6b86b273ff34fce19d6b804eff5a3f5747ada4eaa22f1d49c01e52ddb7875b4b *qemu/debian_riscv64/VERSION
+7b6a36b0c2cdd85f3bbee4156de6c65a6f9b01b6bdf95eeb327b6182807ef111 *qemu/debian_riscv64/disk.qcow2
+45755dd1fec10b7d5d67156bf806d3069a9285d18c5b16f06050e5115cc93f18 *qemu/debian_riscv64/kernel
+16a1077672903239e2e4451e6898f9d813b369f9ceede30da8937a1361a050d7 *qemu/debian_riscv64/fw_jump.elf

package/src/abi_arm32.cc

@@ -11,7 +11,7 @@
 // You should have received a copy of the GNU Affero General Public License
 // along with this program. If not, see https://www.gnu.org/licenses/.
 
-#if defined(__arm__)
+#if defined(__arm__) || (defined(__M_ARM) && !defined(_M_ARM64))
 
 #include "vendor/libcc/libcc.hh"
 #include "ffi.hh"
@@ -19,6 +19,8 @@
 #include "util.hh"
 
 #include <napi.h>
+#include <signal.h>
+#include <setjmp.h>
 
 namespace RG {
 
@@ -37,11 +39,19 @@ extern "C" uint64_t ForwardCallXGG(const void *func, uint8_t *sp);
 extern "C" float ForwardCallXF(const void *func, uint8_t *sp);
 extern "C" HfaRet ForwardCallXDDDD(const void *func, uint8_t *sp);
 
+static inline int IsHFA(const TypeInfo *type)
+{
+#ifdef __ARM_PCS_VFP
+    return IsHFA(type, 1, 4);
+#else
+    return false;
+#endif
+}
+
 bool AnalyseFunction(InstanceData *, FunctionInfo *func)
 {
-    if (IsHFA(func->ret.type, 1, 4)) {
-        func->ret.vec_count = func->ret.type->members.len *
-                              (func->ret.type->members[0].type->size / 4);
+    if (int hfa = IsHFA(func->ret.type); hfa) {
+        func->ret.vec_count = hfa;
     } else if (func->ret.type->primitive != PrimitiveKind::Record ||
                func->ret.type->size <= 4) {
         func->ret.gpr_count = (func->ret.type->size > 4) ? 2 : 1;
@@ -84,13 +94,12 @@ bool AnalyseFunction(InstanceData *, FunctionInfo *func)
                 }
             } break;
             case PrimitiveKind::Record: {
-                if (IsHFA(param.type, 1, 4)) {
-                    int vec_count = (int)(param.type->members.len *
-                                          param.type->members[0].type->size / 4);
+                int hfa = IsHFA(param.type);
 
-                    if (vec_count <= vec_avail) {
-                        param.vec_count = vec_count;
-                        vec_avail -= vec_count;
+                if (hfa) {
+                    if (hfa <= vec_avail) {
+                        param.vec_count = hfa;
+                        vec_avail -= hfa;
                     } else {
                         vec_avail = 0;
                         started_stack = true;
@@ -112,8 +121,13 @@ bool AnalyseFunction(InstanceData *, FunctionInfo *func)
             case PrimitiveKind::Array: { RG_UNREACHABLE(); } break;
             case PrimitiveKind::Float32:
             case PrimitiveKind::Float64: {
-                Size need = param.type->size / 4;
+#ifdef __ARM_PCS_VFP
                 bool vfp = !param.variadic;
+#else
+                bool vfp = false;
+#endif
+
+                Size need = param.type->size / 4;
 
                 if (vfp) {
                     if (need <= vec_avail) {
@@ -374,8 +388,7 @@ bool CallData::Prepare(const Napi::CallbackInfo &info)
         }
     }
 
-    stack = MakeSpan(mem->stack.end(), old_stack_mem.end() - mem->stack.end());
-    heap = MakeSpan(old_heap_mem.ptr, mem->heap.ptr - old_heap_mem.ptr);
+    sp = mem->stack.end();
 
     return true;
 }
@@ -384,8 +397,8 @@ void CallData::Execute()
 {
 #define PERFORM_CALL(Suffix) \
         ([&]() { \
-            auto ret = (func->forward_fp ? ForwardCallX ## Suffix(func->func, stack.ptr) \
-                                         : ForwardCall ## Suffix(func->func, stack.ptr)); \
+            auto ret = (func->forward_fp ? ForwardCallX ## Suffix(func->func, sp) \
+                                         : ForwardCall ## Suffix(func->func, sp)); \
             return ret; \
         })()
 
@@ -407,7 +420,7 @@ void CallData::Execute()
         case PrimitiveKind::Record: {
             if (func->ret.vec_count) {
                 HfaRet ret = PERFORM_CALL(DDDD);
-                memcpy_safe(&result.buf, &ret, RG_SIZE(ret));
+                memcpy(&result.buf, &ret, RG_SIZE(ret));
             } else {
                 result.u64 = PERFORM_CALL(GG);
             }

package/src/abi_arm32_fwd.S

@@ -7,7 +7,7 @@
 // but WITHOUT ANY WARRANTY; without even the implied warranty of
 // MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 // GNU Affero General Public License for more details.
-
+//
 // You should have received a copy of the GNU Affero General Public License
 // along with this program. If not, see https://www.gnu.org/licenses/.