Merge pull request #11 from Koromix/master

Use accurate benchmark for Koffi
zhangyuang · Oct 26, 2023 · 1ad939c · 1ad939c
2 parents 1776184 + 3b139b3
commit 1ad939c
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/bench/bench.js b/bench/bench.js
@@ -12,10 +12,14 @@ open({
   library: 'libsum',
   path: dynamicLib
 })
-const libm = ffi.Library('libsum', {
+const libm = ffi.Library('./libsum', {
   'sum': ['int', ['int', 'int']],
   concatenateStrings: ['string', ['string', 'string']],
 });
+const koffi_libm = {
+  sum: koffilib.func('int sum(int a, int b)'),
+  concatenateStrings: koffilib.func('const char *concatenateStrings(const char *str1, const char *str2)')
+};
 
 async function run() {
   await b.suite(
@@ -25,10 +29,8 @@ async function run() {
       libm.concatenateStrings("foo", "bar");
     }),
     b.add('koffi', () => {
-      const sum = koffilib.func('int sum(int a, int b)');
-      const concatenateStrings = koffilib.func('const char *concatenateStrings(const char *str1, const char *str2)');
-      sum(1, 2)
-      concatenateStrings("foo", "bar")
+      koffi_libm.sum(1, 2)
+      koffi_libm.concatenateStrings("foo", "bar")
     }),
     b.add('ffi-rs', () => {
       load({

diff --git a/cpp/sum.cpp b/cpp/sum.cpp
@@ -3,7 +3,6 @@
 #include <iostream>
 #include <string>
 #include <vector>
-#include <xlocale/_stdio.h>
 
 extern "C" int sum(int a, int b) { return a + b; }