Merge pull request #4 from mrsuh/arm64-support

Added arm64 support

Author: mrsuh

.github/workflows/tests.yml

@@ -16,6 +16,7 @@ jobs:
                     - "8.0"
                     - "8.1"
                     - "8.2"
+                    - "8.3"
                 operating-system:
                     - "ubuntu-latest"
 

Makefile

@@ -1,19 +1,20 @@
-UNAME := $(shell uname -s)
+OS := $(shell uname -s)
+PLATFORM := $(shell uname -m)
 HFOLDERS := -I php-src -I php-src/Zend -I php-src/main -I php-src/TSRM -I php-src/ext/spl
-COMMAND := @echo Invalid operation system: $(UNAME)
+COMMAND := @echo Invalid operation system: $(OS)
 
 ifeq ($(DEBUG), 1)
 	DEBUGFLAG := -D DEBUG
 else
 	DEBUGFLAG :=
 endif
 
-ifeq ($(UNAME), Linux)
-	COMMAND := g++ -O3 -fPIC -shared $(HFOLDERS) $(DEBUGFLAG) -o library/ffi_linux.so library/ffi.cpp
+ifeq ($(OS), Linux)
+	COMMAND := g++ -O3 -fPIC -shared $(HFOLDERS) $(DEBUGFLAG) -o library/ffi_linux_$(PLATFORM).so library/ffi.cpp
 endif
 
-ifeq ($(UNAME), Darwin)
-	COMMAND := clang -shared -undefined dynamic_lookup $(HFOLDERS) $(DEBUGFLAG) -o library/ffi_darwin.dylib library/ffi.cpp
+ifeq ($(OS), Darwin)
+	COMMAND := clang -shared -undefined dynamic_lookup $(HFOLDERS) $(DEBUGFLAG) -o library/ffi_darwin_$(PLATFORM).dylib library/ffi.cpp
 endif
 
 all:

README.md

@@ -12,7 +12,7 @@ It doesn't take into calculate the memory of handlers/functions/etc.
 
 ### Requirements
 * PHP >= 7.4 (with FFI)
-* Linux(x86_64) / Darwin(x86_64)
+* Linux(x86_64/aarch64) / Darwin(x86_64/arm64)
 
 ### How to install
 ```bash
@@ -104,9 +104,17 @@ docker run -it --rm --name my-running-script -v "$PWD":/app image-php-var-sizeof
 
 ### How to compile library
 ```bash
+git submodule update --init --recursive
 cd php-src
 ./buildconf
-./configure
+./configure --with-ffi --with-iconv=/opt/homebrew/opt/libiconv
 cd ..
 make DEBUG=1
 ```
+
+```bash
+docker build -t php-var-sizeof -f Dockerfile-compile .
+docker run -it -v $(pwd)/library:/code/library php-var-sizeof bash
+cd /code
+make DEBUG=1
+```

library/ffi.h

@@ -0,0 +1,9 @@
+#include "zend_types.h"
+
+int zval_sizeof(zval *zv_ptr);
+
+int zval_class_sizeof(zval *zv_ptr);
+
+int properties_sizeof(zval *srcProperty, int count);
+
+zval *get_zval_by_name(char *name);

library/ffi_darwin_arm64.dylib

@@ -12,27 +12,27 @@ public static function init(): void
             return;
         }
 
-        if (\php_uname('m') !== 'x86_64') {
-            throw new \RuntimeException('Unsupported machine type');
-        }
-
-        switch (\php_uname('s')) {
-            case 'Linux':
-                $libraryFileName = 'ffi_linux.so';
-                break;
-            case 'Darwin':
-                $libraryFileName = 'ffi_darwin.dylib';
-                break;
-            default:
-                throw new \RuntimeException('Unsupported operating system');
+        $platform = strtolower(\php_uname('m'));
+        $operatingSystem = strtolower(\php_uname('s'));
+
+        $libraryFilePath = sprintf(
+            "%s/../library/ffi_%s_%s.%s",
+            __DIR__,
+            $operatingSystem,
+            $platform,
+            $operatingSystem === 'linux' ? 'so' : 'dylib'
+        );
+
+        if(!is_file($libraryFilePath)) {
+            throw new \RuntimeException(sprintf('Unsupported system "%s(%s)"', $operatingSystem, $platform));
         }
 
         self::$libc = \FFI::cdef(
             "
             int var_sizeof(char *name);
             int var_class_sizeof(char *name);
             ",
-            __DIR__ . "/../library/" . $libraryFileName);
+            $libraryFilePath);
     }
 
     /**